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

Add default serialization for Idents #63853

Merged
merged 1 commit into from
Aug 28, 2019

Conversation

matthewjasper
Copy link
Contributor

Also add tests for -Zast-json and -Zast-json-noexpand

closes #63728

@rust-highfive
Copy link
Collaborator

r? @estebank

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 24, 2019
@rust-highfive

This comment has been minimized.

@matthewjasper matthewjasper force-pushed the test-ast-serialization branch from 9cb7414 to 0bde766 Compare August 24, 2019 19:08
@estebank
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Aug 26, 2019

📌 Commit 0bde766 has been approved by estebank

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 26, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 27, 2019
…n, r=estebank

Add default serialization for `Ident`s

Also add tests for `-Zast-json` and `-Zast-json-noexpand`

closes rust-lang#63728
@Centril
Copy link
Contributor

Centril commented Aug 27, 2019

Failed in #63936 (comment), @bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 27, 2019
Add tests for -Zast-json and -Zast-json-noexpand, which need this impl.
@matthewjasper matthewjasper force-pushed the test-ast-serialization branch from 0bde766 to 3d71803 Compare August 27, 2019 22:02
@matthewjasper
Copy link
Contributor Author

@bors r=estebank

@bors
Copy link
Contributor

bors commented Aug 28, 2019

📌 Commit 3d71803 has been approved by estebank

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 28, 2019
@bors
Copy link
Contributor

bors commented Aug 28, 2019

⌛ Testing commit 3d71803 with merge c422372...

bors added a commit that referenced this pull request Aug 28, 2019
Add default serialization for `Ident`s

Also add tests for `-Zast-json` and `-Zast-json-noexpand`

closes #63728
@bors
Copy link
Contributor

bors commented Aug 28, 2019

☀️ Test successful - checks-azure
Approved by: estebank
Pushing c422372 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 28, 2019
@bors bors merged commit 3d71803 into rust-lang:master Aug 28, 2019
@matthewjasper matthewjasper deleted the test-ast-serialization branch August 29, 2019 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
5 participants