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 outdated option to -Zcheck-cfg warnings #12884

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

Urgau
Copy link
Member

@Urgau Urgau commented Oct 27, 2023

Forgot to remove those in #12845.

Note: Would it make sense to have feature-gate tests for those? and if so what would be the recommended way?

@rustbot
Copy link
Collaborator

rustbot commented Oct 27, 2023

r? @epage

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

@rustbot rustbot added A-build-scripts Area: build.rs scripts S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 27, 2023
@epage
Copy link
Contributor

epage commented Oct 27, 2023

Note: Would it make sense to have feature-gate tests for those? and if so what would be the recommended way?

We regularly have feature gate tests. In check_cfg.rs, just add a test with a build script and without a feature flag that verifies the warning is in the output.

@epage
Copy link
Contributor

epage commented Oct 27, 2023

Would you like to add the tests as part of this or should I merge it and you'll add them later?

@Urgau
Copy link
Member Author

Urgau commented Oct 27, 2023

I can do it in this PR, let's me quickly add them.

@epage
Copy link
Contributor

epage commented Oct 27, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 27, 2023

📌 Commit e396904 has been approved by epage

It is now in the queue for this repository.

@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 27, 2023
@bors
Copy link
Collaborator

bors commented Oct 27, 2023

⌛ Testing commit e396904 with merge de54757...

@bors
Copy link
Collaborator

bors commented Oct 27, 2023

☀️ Test successful - checks-actions
Approved by: epage
Pushing de54757 to master...

@bors bors merged commit de54757 into rust-lang:master Oct 27, 2023
18 of 20 checks passed
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 28, 2023
Update cargo

8 commits in df3509237935f9418351b77803df7bc05c009b3d..708383d620e183a9ece69b8fe930c411d83dee27
2023-10-24 23:09:01 +0000 to 2023-10-27 21:09:26 +0000
- feat(doc): Print the generated docs links (rust-lang/cargo#12859)
- feat(toml): Allow version-less manifests (rust-lang/cargo#12786)
- Remove outdated option to `-Zcheck-cfg` warnings (rust-lang/cargo#12884)
- Remove duplicate binaries during install (rust-lang/cargo#12868)
- refactor(shell): Write at once rather than in fragments (rust-lang/cargo#12880)
- docs(ref): Link to docs.rs metadata table (rust-lang/cargo#12879)
- docs(contrib): Describe how to add a new package (rust-lang/cargo#12878)
- move up looking at index summary enum (rust-lang/cargo#12749)

r? ghost
@ehuss ehuss added this to the 1.75.0 milestone Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-build-scripts Area: build.rs scripts 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.

5 participants