Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add iterators to doubly::List #4

Open
hawkw opened this issue Jan 20, 2018 · 7 comments
Open

Add iterators to doubly::List #4

hawkw opened this issue Jan 20, 2018 · 7 comments
Labels
enhancement good first issue
Milestone
intruder_alarm 0.1.0

Comments

@hawkw
Copy link
Member

hawkw commented Jan 20, 2018

No description provided.

@hawkw
Copy link
Member Author

hawkw commented Jan 20, 2018

N.B. that a doubly linked list's iterators can be DoubleEndedIterator and ExactSizeIterator.

@hawkw
Copy link
Member Author

hawkw commented Jan 20, 2018

This could be a reasonably good first issue for someone with some prior Rust experience; it's a bit larger in scope than some of the other "good first issue" issues, but it's fairly self-contained.

@hawkw hawkw added this to the intruder_alarm 0.1.0 milestone Jan 20, 2018
@JJJollyjim
Copy link

I'm going to try and take a shot at this

@hawkw
Copy link
Member Author

hawkw commented Jan 21, 2018

@JJJollyjim excellent! I've invited you to become a repo collaborator so I can assign this issue to you.

CONTRIBUTING.md has some introductory information on how to contribute to this repository. Do let me know if you have any questions or need any advice on implementing this issue!

@JJJollyjim
Copy link

Hi again: I got lost in lifetime hell so didn't get it done today when I expected.

I go back to work tomorrow so probably can't get it done for quite a while: anyone else can feel free to grab this issue.

@hawkw
Copy link
Member Author

hawkw commented Jan 22, 2018

@JJJollyjim that's fine! If you want to, push up a branch for what you've got and I can take a peek at it?

@hawkw hawkw mentioned this issue Jan 28, 2018
Merged
@hawkw
Copy link
Member Author

hawkw commented Jan 28, 2018

@JJJollyjim, if you're still interested in working on this issue, I just opened PR #31, which may be relevant to your interests here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue
Projects
None yet
Milestone
intruder_alarm 0.1.0
Development

No branches or pull requests
2 participants
@JJJollyjim @hawkw