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

Remove unused AstBuilder #7210

Merged
merged 1 commit into from
Aug 4, 2019

Conversation

Mark-Simulacrum
Copy link
Member

This was removed in a recent rustc PR (rust-lang/rust#63146), replaced with inherent impls.

@rust-highfive
Copy link

r? @ehuss

(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 4, 2019
@ehuss
Copy link
Contributor

ehuss commented Aug 4, 2019

Thanks!
@bors r+

@bors
Copy link
Collaborator

bors commented Aug 4, 2019

📌 Commit aba2aa2 has been approved by ehuss

@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 4, 2019
@bors
Copy link
Collaborator

bors commented Aug 4, 2019

⌛ Testing commit aba2aa2 with merge fe7b427...

bors added a commit that referenced this pull request Aug 4, 2019
Remove unused AstBuilder

This was removed in a recent rustc PR (rust-lang/rust#63146), replaced with inherent impls.
@bors
Copy link
Collaborator

bors commented Aug 4, 2019

☀️ Test successful - checks-azure
Approved by: ehuss
Pushing fe7b427 to master...

@bors bors merged commit aba2aa2 into rust-lang:master Aug 4, 2019
bors added a commit to rust-lang/rust that referenced this pull request Aug 8, 2019
Update cargo

9 commits in 26092da337b948719549cd5ed3d1051fd847afd7..42a8c0adf91323c01228268c651aef5366b25b69
2019-07-31 23:24:32 +0000 to 2019-08-07 20:41:07 +0000
- Add support for customizing JSON diagnostics from Cargo (rust-lang/cargo#7214)
- Bump rustfix (rust-lang/cargo#7221)
- Fix remap-path-prefix from failing. (rust-lang/cargo#7219)
- Clean up build script stuff and documentation. (rust-lang/cargo#7215)
- Remove debug panic in package-features. (rust-lang/cargo#7213)
- Fix an old test. (rust-lang/cargo#7208)
- Remove unused AstBuilder (rust-lang/cargo#7210)
- Revert "Release a jobserver token while locking a file" (rust-lang/cargo#7201)
- improve error message for unmatched prerelease dependencies (rust-lang/cargo#7191)
matklad added a commit to matklad/rust that referenced this pull request Sep 21, 2019
The mentioned Cargo test is fixed in rust-lang/cargo#7210
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?
@ehuss ehuss added this to the 1.38.0 milestone Feb 6, 2022
@Mark-Simulacrum Mark-Simulacrum deleted the remove-useless-builder branch July 10, 2022 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: 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