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

Cfg hide more conditions for core and alloc #89668

Merged
merged 2 commits into from
Oct 9, 2021
Merged

Conversation

Urgau
Copy link
Member

@Urgau Urgau commented Oct 8, 2021

Fixes #89663

Before:
image

After:
image

Same for alloc

r? @GuillaumeGomez

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 8, 2021
@GuillaumeGomez
Copy link
Member

Thanks!

r=me once CI pass

@KamilaBorowska
Copy link
Contributor

This also affects alloc crate I believe.

@GuillaumeGomez
Copy link
Member

It does indeed: https://doc.rust-lang.org/nightly/alloc/index.html

@Urgau: Please add the fix there as well.

@Urgau Urgau changed the title Cfg hide more conditions for core Cfg hide more conditions for core and alloc Oct 8, 2021
@Urgau
Copy link
Member Author

Urgau commented Oct 8, 2021

It does indeed: https://doc.rust-lang.org/nightly/alloc/index.html

@Urgau: Please add the fix there as well.

Done

@Urgau
Copy link
Member Author

Urgau commented Oct 8, 2021

Let's try that.

@bors r=GuillaumeGomez rollup

EDIT: Well, didn't worked as planned. Sorry for the noise.

@bors
Copy link
Contributor

bors commented Oct 8, 2021

@Urgau: 🔑 Insufficient privileges: Not in reviewers

@bors
Copy link
Contributor

bors commented Oct 8, 2021

@Urgau: 🔑 Insufficient privileges: not in try users

@jyn514
Copy link
Member

jyn514 commented Oct 8, 2021

@bors r=GuillaumeGomez rollup

@bors
Copy link
Contributor

bors commented Oct 8, 2021

📌 Commit 0a03ec4 has been approved by GuillaumeGomez

@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 Oct 8, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 8, 2021
…laumeGomez

Rollup of 6 pull requests

Successful merges:

 - rust-lang#86506 (Don't normalize xform_ret_ty during method candidate assembly )
 - rust-lang#89538 (Make rustdoc not highlight `->` and `=>` as operators)
 - rust-lang#89649 (clippy::complexity fixes)
 - rust-lang#89668 (Cfg hide more conditions for core and alloc)
 - rust-lang#89669 (Remove special-casing of never primitive in rustdoc-json-types)
 - rust-lang#89672 (Remove unwrap_or! macro)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 1c35968 into rust-lang:master Oct 9, 2021
@rustbot rustbot added this to the 1.57.0 milestone Oct 9, 2021
@Urgau Urgau deleted the core-cfg-hide branch May 5, 2023 16:45
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
7 participants