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.

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

move code to the 2018 edition #19

Merged
merged 2 commits into from
Sep 14, 2018
Merged

move code to the 2018 edition #19

merged 2 commits into from
Sep 14, 2018

Conversation

japaric
Copy link
Member

@japaric japaric commented Sep 5, 2018

most code changes are minor

the annoying part of using the 2018 edition right now is that there is no easy
way to create new 2018 project (Cargo still defaults to 2015 projects; see
rust-lang/cargo#5980) and there's no visible documentation on how to move a new
project to the 2018 edition (see rust-lang/edition-guide#102) so I have
included the required changes in the preface -- I hope people won't miss that
part.

r? @rust-embedded/resources (anyone)

closes #1

@japaric japaric requested a review from a team as a code owner September 5, 2018 16:14
@thejpster
Copy link

LGTM

this commit removes features gate from the code and from Cargo.toml

it also makes use of the new `--edition` flag to simplify the instructions for
creating a new project
@japaric
Copy link
Member Author

japaric commented Sep 14, 2018

Latest commit prepares this book for the upcoming beta release.

r? @rust-embedded/resources

Copy link
Contributor

@therealprof therealprof left a comment

Choose a reason for hiding this comment

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

LGTM

@japaric
Copy link
Member Author

japaric commented Sep 14, 2018

@therealprof Thanks!

bors r+

bors bot added a commit that referenced this pull request Sep 14, 2018
19: move code to the 2018 edition r=japaric a=japaric

most code changes are minor

the annoying part of using the 2018 edition right now is that there is no easy
way to create new 2018 project (Cargo still defaults to 2015 projects; see
rust-lang/cargo#5980) and there's no visible documentation on how to move a new
project to the 2018 edition (see rust-lang/edition-guide#102) so I have
included the required changes in the preface -- I hope people won't miss that
part.

r? @rust-embedded/resources (anyone)

closes #1

Co-authored-by: Jorge Aparicio <jorge@japaric.io>
@bors
Copy link
Contributor

bors bot commented Sep 14, 2018

Build succeeded

@bors bors bot merged commit 508802f into master Sep 14, 2018
@bors bors bot deleted the 2018 branch September 14, 2018 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port code to the 2018 edition
3 participants