Skip to content

/walkdir

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.

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

Use `?` in docs instead of unwrapping #25

Closed
KodrAus opened this Issue Jun 14, 2017 · 0 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@KodrAus
@KodrAus
Copy link
Contributor

KodrAus commented Jun 14, 2017

Relevant API Guideline

Instead of calling unwrap in the doc examples, we should use the ? operator:

  • examples in crate root
  • examples on WalkDir type
  • examples on WalkDirIterator.skip_current_dir
  • examples on WalkDirIterator.filter_entry

@KodrAus KodrAus referenced this issue Jun 14, 2017

Closed

Tracking issue for libz blitz evaluation of walkdir #47

22 of 22 tasks complete

msehnout added a commit to msehnout/walkdir that referenced this issue Jun 16, 2017

@msehnout
Use `?` in docs instead of unwrapping 
Fixes: BurntSushi#25
1d43a33

@msehnout msehnout referenced this issue Jun 16, 2017

Merged

Use `?` in docs instead of unwrapping #52

msehnout added a commit to msehnout/walkdir that referenced this issue Jun 19, 2017

@msehnout
Use `?` in docs instead of unwrapping 
Fixes: BurntSushi#25
b984671

msehnout added a commit to msehnout/walkdir that referenced this issue Jun 19, 2017

@msehnout
Use `?` in docs instead of unwrapping 
Fixes: BurntSushi#25
b2e3670

msehnout added a commit to msehnout/walkdir that referenced this issue Jun 30, 2017

@msehnout
Use `?` in docs instead of unwrapping 
Fixes: BurntSushi#25
672499f

@BurntSushi BurntSushi closed this in #52 Jun 30, 2017

BurntSushi added a commit that referenced this issue Jun 30, 2017

@msehnout @BurntSushi
Use `?` in docs instead of unwrapping 
Fixes: #25
Loading status checks…
938b0fa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.