Skip to content

reduce rustc_attr usage in places - #70526

Merged
bors merged 1 commit into
rust-lang:masterfrom
Centril:less-attr
Mar 29, 2020
Merged

reduce rustc_attr usage in places#70526
bors merged 1 commit into
rust-lang:masterfrom
Centril:less-attr

Conversation

@Centril

Copy link
Copy Markdown
Contributor

This cleans up some unused rustc_attr dependencies.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @cramertj

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 29, 2020
@eddyb

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Mar 29, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 59a6970 has been approved by eddyb

@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 Mar 29, 2020
@bors

bors commented Mar 29, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 59a6970 with merge 645a105a1c544559c81e57a6840f5cb934ff962d...

@CentrilCentril mentioned this pull request Mar 29, 2020
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors retry

bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 29, 2020
Rollup of 3 pull requests
Successful merges:
- rust-lang#70140 (Add Result<Result<T, E>, E>::flatten -> Result<T, E>)
- rust-lang#70526 (reduce `rustc_attr` usage in places)
- rust-lang#70527 (Update LLVM submodule)
Failed merges:
r? @ghost
@bors
bors merged commit b4491e5 into rust-lang:masterMar 29, 2020
@Centril
Centril deleted the less-attr branch March 29, 2020 22:30
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.

5 participants

@Centril@rust-highfive@eddyb@bors@cramertj