Skip to content

Type annotations for deque should include | null #11

@jdunck

Description

@jdunck

These methods could return null:

  • popFront
  • popBack
  • front
  • back

The docs seem to indicate they do but the type annotations do not.

I think we should either add | null here, or we should raise when the deque is empty. I'd actually prefer that, since right now, Deque can't store a null-value.

What is your preference?

Metadata

Metadata

Labels

bugSomething isn't working

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions