Skip to content

Expose fs::create_dir_all #111

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

Merged
merged 4 commits into from
Aug 26, 2019
Merged

Conversation

dfrankland
Copy link
Contributor

I think this function was accidentally unimported originally, but this PR fixes that 😄

@skade skade self-assigned this Aug 26, 2019
@skade
Copy link
Collaborator

skade commented Aug 26, 2019

Thanks! That indeed looks like a mistake. Could you also add a line to the CHANGELOG?

CHANGELOG.md Outdated
@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://book.async.rs/overview
- Many small changes in the book, mostly typos
- Documentation fixes correcting examples
- Now works with recent nightly with stabilised async/await (> 2019-08-21)
- Expose `fs::create_dir_all`
Copy link
Collaborator

@skade skade Aug 26, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, sorry ;). It should go in the "Unreleased" section, 0.99.4 has already been released.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woops! The headings threw me for a loop there.

@dfrankland dfrankland force-pushed the expose-fs-create_dir_all branch from a479204 to 2fc6610 Compare August 26, 2019 19:55
@skade
Copy link
Collaborator

skade commented Aug 26, 2019

Thanks, I'll wait for CI to finish.

@skade skade added the accepted Accepted changes and pull requests that can be implemented or merged label Aug 26, 2019
@skade
Copy link
Collaborator

skade commented Aug 26, 2019

And done, thanks!

@skade skade merged commit a14b11c into async-rs:master Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Accepted changes and pull requests that can be implemented or merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants