Skip to content

Upgrade to 2018 Edition#7

Merged
Hoverbear merged 5 commits intomasterfrom
edition
Dec 23, 2018
Merged

Upgrade to 2018 Edition#7
Hoverbear merged 5 commits intomasterfrom
edition

Conversation

@Hoverbear
Copy link
Copy Markdown
Contributor

Follows from tikv/rfcs#7.

Copy link
Copy Markdown
Member

@sunxiaoguang sunxiaoguang left a comment

Choose a reason for hiding this comment

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

Since Edition 2018 is officially released, how about we merge this pull request now?

Comment thread examples/transaction.rs
Comment thread rust-toolchain Outdated
Comment thread src/raw.rs Outdated
Signed-off-by: Hoverbear <operator@hoverbear.org>
Signed-off-by: Hoverbear <operator@hoverbear.org>
Signed-off-by: Hoverbear <operator@hoverbear.org>
Signed-off-by: Hoverbear <operator@hoverbear.org>
Signed-off-by: Hoverbear <operator@hoverbear.org>
Copy link
Copy Markdown
Contributor

@siddontang siddontang left a comment

Choose a reason for hiding this comment

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

LGTM

PTAL @brson

@sunxiaoguang
Copy link
Copy Markdown
Member

sunxiaoguang commented Dec 23, 2018

Can we merge this now? And I will be able to upgrade the code I'm working on to 2018 edition.

@Hoverbear Hoverbear merged commit 5ed3327 into master Dec 23, 2018
@Hoverbear
Copy link
Copy Markdown
Contributor Author

Let's do it then. :)

@sunxiaoguang
Copy link
Copy Markdown
Member

Let's do it then. :)

Cool:)

@Hoverbear Hoverbear deleted the edition branch December 27, 2018 19:47
sunxiaoguang pushed a commit that referenced this pull request Jan 11, 2019
* Bump to 2018 edition.

Signed-off-by: Hoverbear <operator@hoverbear.org>

* Add toolchain

Signed-off-by: Hoverbear <operator@hoverbear.org>

* Fix examples

Signed-off-by: Hoverbear <operator@hoverbear.org>

* fmt and lint

Signed-off-by: Hoverbear <operator@hoverbear.org>

* We can use stable now

Signed-off-by: Hoverbear <operator@hoverbear.org>
Signed-off-by: Xiaoguang Sun <sunxiaoguang@zhihu.com>
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.

4 participants