Skip to content

fully remove AstBuilder - #64664

Merged
bors merged 1 commit into
rust-lang:masterfrom
matklad:remove-ast-builder
Sep 22, 2019
Merged

fully remove AstBuilder#64664
bors merged 1 commit into
rust-lang:masterfrom
matklad:remove-ast-builder

Conversation

@matklad

@matkladmatklad commented Sep 21, 2019

Copy link
Copy Markdown
Contributor

The mentioned Cargo test is fixed in rust-lang/cargo#7210

I think this can be removed now?

The mentioned Cargo test is fixed in rust-lang/cargo#7210
@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 21, 2019
@Mark-Simulacrum

Copy link
Copy Markdown
Member

Yes, I'm pretty sure we've updated Cargo since then so should be fine to remove...

@bors r+ rollup

@bors

bors commented Sep 21, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 53a6a2f has been approved by Mark-Simulacrum

@borsbors 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 Sep 21, 2019
Centril added a commit to Centril/rust that referenced this pull request Sep 21, 2019
…Simulacrum
fully remove AstBuilder
The mentioned Cargo test is fixed in rust-lang/cargo#7210
I think this can be removed now?
@CentrilCentril mentioned this pull request Sep 21, 2019
bors added a commit that referenced this pull request Sep 22, 2019
Rollup of 9 pull requests
Successful merges:
- #63907 (Add explanation to type mismatch involving type params and assoc types)
- #64615 (rustbuild: Turn down compression on exe installers)
- #64617 (rustbuild: Turn down compression on msi installers)
- #64618 (rustbuild: Improve output of `dist` step)
- #64619 (Fixes#63962. Hint about missing tuple parentheses in patterns)
- #64634 (Update to LLVM 9.0.0)
- #64635 (Allow using fn pointers in const fn with unleash miri)
- #64660 (unify errors for tuple/struct variants)
- #64664 (fully remove AstBuilder)
Failed merges:
r? @ghost
@bors
bors merged commit 53a6a2f into rust-lang:masterSep 22, 2019
@matklad
matklad deleted the remove-ast-builder branch September 22, 2019 06:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@matklad@Mark-Simulacrum@bors@rust-highfive