Skip to content

Accept trailing comma in cfg_attr #54581

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Sep 27, 2018
Merged

Conversation

petrochenkov
Copy link
Contributor

Fixes #54463 (stable-to-beta regression)

@petrochenkov petrochenkov added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Sep 26, 2018
@rust-highfive
Copy link
Contributor

r? @eddyb

(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 Sep 26, 2018
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Sep 26, 2018

📌 Commit 2b3e16b has been approved by alexcrichton

@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 Sep 26, 2018
@pietroalbini pietroalbini added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Sep 26, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Sep 27, 2018
Accept trailing comma in `cfg_attr`

Fixes rust-lang#54463 (stable-to-beta regression)
@bors
Copy link
Collaborator

bors commented Sep 27, 2018

⌛ Testing commit 2b3e16b with merge e999ebd...

bors added a commit that referenced this pull request Sep 27, 2018
Accept trailing comma in `cfg_attr`

Fixes #54463 (stable-to-beta regression)
@bors
Copy link
Collaborator

bors commented Sep 27, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing e999ebd to master...

@bors bors merged commit 2b3e16b into rust-lang:master Sep 27, 2018
@oli-obk oli-obk added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Oct 2, 2018
@pnkfelix
Copy link
Member

pnkfelix commented Oct 2, 2018

re-adding beta-nominated so that this will (hopefully) be enqueued for backport

@pnkfelix pnkfelix added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Oct 2, 2018
@pietroalbini pietroalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Oct 2, 2018
bors added a commit that referenced this pull request Oct 2, 2018
[beta] Rollup backports

Merged and approved:

* #54650: Don't lint non-extern-prelude extern crate's in Rust 2018.
* #54338: Use full name to identify a macro in a `FileName`.
* #54518: resolve: Do not block derive helper resolutions on single import resolutions
* #54581: Accept trailing comma in `cfg_attr`

r? @ghost
bors added a commit that referenced this pull request Oct 2, 2018
[beta] Rollup backports

Merged and approved:

* #54650: Don't lint non-extern-prelude extern crate's in Rust 2018.
* #54338: Use full name to identify a macro in a `FileName`.
* #54518: resolve: Do not block derive helper resolutions on single import resolutions
* #54581: Accept trailing comma in `cfg_attr`

r? @ghost
@petrochenkov petrochenkov deleted the cfgattr branch June 5, 2019 16:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants