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.

Sign up for GitHub

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

Jump to bottom

[cfg_match] Document the use of expressions. #135625

Merged
merged 1 commit into from
Jan 29, 2025
Merged

[cfg_match] Document the use of expressions. #135625

merged 1 commit into from
Jan 29, 2025

Conversation

c410-f3r
Copy link
Contributor

@c410-f3r c410-f3r commented Jan 17, 2025 •
edited
Loading

cc #115585

Adds documentation to this new feature introduced in #133720.

@rustbot
Copy link
Collaborator

rustbot commented Jan 17, 2025

r? @joboet

rustbot has assigned @joboet.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jan 17, 2025
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@ehuss
Copy link
Contributor

ehuss commented Jan 17, 2025

Can you remove all of the submodule updates?

@c410-f3r c410-f3r force-pushed the cfg-match-foo-bar-baz branch from 90608ee to 5844eac Compare January 17, 2025 15:30
@c410-f3r
Copy link
Contributor Author

c410-f3r commented Jan 21, 2025 •
edited
Loading

r? T-libs

@rustbot rustbot assigned tgross35 and jhpratt and unassigned joboet and tgross35 Jan 21, 2025
tgross35
tgross35 reviewed Jan 21, 2025
View reviewed changes
library/core/src/macros/mod.rs Outdated
@@ -313,6 +313,17 @@ pub macro cfg_match {
/// }
/// }
/// ```
///
/// If desired, it is possible to return expressions through the use of surrounding braces.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// If desired, it is possible to return expressions through the use of surrounding braces.
/// If desired, it is possible to return expressions through the use of surrounding braces:

Nit, colon to introduce an example

@tgross35
Copy link
Contributor

One nit and squash please, then r=me

@c410-f3r c410-f3r force-pushed the cfg-match-foo-bar-baz branch from 5844eac to 587b9c6 Compare January 21, 2025 11:45
@jhpratt
Copy link
Member

jhpratt commented Jan 21, 2025

@bors r=tgross35,jhpratt rollup

@bors
Copy link
Contributor

bors commented Jan 21, 2025

📌 Commit 587b9c6 has been approved by tgross35,jhpratt

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 Jan 21, 2025
jieyouxu added a commit to jieyouxu/rust that referenced this pull request Jan 21, 2025
@jieyouxu
Rollup merge of rust-lang#135625 - c410-f3r:cfg-match-foo-bar-baz, r=…
f99cae1 
…tgross35,jhpratt

[cfg_match] Document the use of expressions.

cc rust-lang#115585

Adds documentation to this new feature introduced in rust-lang#133720.
@jieyouxu jieyouxu mentioned this pull request Jan 21, 2025
Closed
jieyouxu added a commit to jieyouxu/rust that referenced this pull request Jan 21, 2025
@jieyouxu
Rollup merge of rust-lang#135625 - c410-f3r:cfg-match-foo-bar-baz, r=…
b4e2fb5 
…tgross35,jhpratt

[cfg_match] Document the use of expressions.

cc rust-lang#115585

Adds documentation to this new feature introduced in rust-lang#133720.
@jieyouxu jieyouxu mentioned this pull request Jan 21, 2025
Closed
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 21, 2025
@bors
Auto merge of rust-lang#135825 - jieyouxu:rollup-bske4m7, r=jieyouxu
dfd5d41 
Rollup of 10 pull requests

Successful merges:

 - rust-lang#132232 (CI: build FreeBSD artifacts on FreeBSD 13.4)
 - rust-lang#135625 ([cfg_match] Document the use of expressions.)
 - rust-lang#135638 (Make it possible to build GCC on CI)
 - rust-lang#135648 (support wasm inline assembly in `naked_asm!`)
 - rust-lang#135707 (Shorten linker output even more when `--verbose` is not present)
 - rust-lang#135750 (Add an example of using `carrying_mul_add` to write wider multiplication)
 - rust-lang#135779 (CI: free disk on linux arm runner)
 - rust-lang#135793 (Ignore `mermaid.min.js`)
 - rust-lang#135810 (Add Kobzol on vacation)
 - rust-lang#135814 (ci: use ghcr buildkit image)

r? `@ghost`
`@rustbot` modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jan 21, 2025
@matthiaskrgr
Rollup merge of rust-lang#135625 - c410-f3r:cfg-match-foo-bar-baz, r=…
0e31bd4 
…tgross35,jhpratt

[cfg_match] Document the use of expressions.

cc rust-lang#115585

Adds documentation to this new feature introduced in rust-lang#133720.
@matthiaskrgr matthiaskrgr mentioned this pull request Jan 21, 2025
Closed
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 21, 2025
@bors
Auto merge of rust-lang#135835 - matthiaskrgr:rollup-pzaa7cn, r=matth…
f77ead0 
…iaskrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#132232 (CI: build FreeBSD artifacts on FreeBSD 13.4)
 - rust-lang#135625 ([cfg_match] Document the use of expressions.)
 - rust-lang#135750 (Add an example of using `carrying_mul_add` to write wider multiplication)
 - rust-lang#135793 (Ignore `mermaid.min.js`)
 - rust-lang#135810 (Add Kobzol on vacation)
 - rust-lang#135816 (Use `structurally_normalize` instead of manual `normalizes-to` goals in alias relate errors)
 - rust-lang#135821 (fix OsString::from_encoded_bytes_unchecked description)

r? `@ghost`
`@rustbot` modify labels: rollup
@matthiaskrgr
Copy link
Member

@bors r-
#135835 (comment)

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 21, 2025
@c410-f3r c410-f3r force-pushed the cfg-match-foo-bar-baz branch from 587b9c6 to 7d9fe91 Compare January 21, 2025 20:54
@c410-f3r
Copy link
Contributor Author

@rustbot review

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 21, 2025
@ehuss
Copy link
Contributor

ehuss commented Jan 21, 2025

Do we know why it choked on the comma?

@c410-f3r
Copy link
Contributor Author

The macro does not support trailing commas at the current time.

@c410-f3r
Copy link
Contributor Author

c410-f3r commented Jan 28, 2025 •
edited
Loading

r? T-libs

@rustbot rustbot assigned joboet, tgross35 and Noratrieb and unassigned jhpratt, joboet and tgross35 Jan 28, 2025
@tgross35
Copy link
Contributor

@bors r=tgross35,jhpratt

Do we know why it choked on the comma?

The macro does not support trailing commas at the current time.

Why was this not a problem during PR CI though?

@bors
Copy link
Contributor

bors commented Jan 28, 2025

📌 Commit 7d9fe91 has been approved by tgross35,jhpratt

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 Jan 28, 2025
Zalathar added a commit to Zalathar/rust that referenced this pull request Jan 29, 2025
@Zalathar
Rollup merge of rust-lang#135625 - c410-f3r:cfg-match-foo-bar-baz, r=…
5317161 
…tgross35,jhpratt

[cfg_match] Document the use of expressions.

cc rust-lang#115585

Adds documentation to this new feature introduced in rust-lang#133720.
@Zalathar Zalathar mentioned this pull request Jan 29, 2025
Closed
@fmease fmease mentioned this pull request Jan 29, 2025
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 29, 2025
@bors
Auto merge of rust-lang#136225 - fmease:rollup-fm7m744, r=fmease
ccc9ba5 
Rollup of 7 pull requests

Successful merges:

 - rust-lang#135625 ([cfg_match] Document the use of expressions.)
 - rust-lang#135902 (Do not consider child bound assumptions for rigid alias)
 - rust-lang#135943 (Rename `Piece::String` to `Piece::Lit`)
 - rust-lang#136104 (Add mermaid graphs of NLL regions and SCCs to polonius MIR dump)
 - rust-lang#136143 (Update books)
 - rust-lang#136147 (ABI-required target features: warn when they are missing in base CPU)
 - rust-lang#136164 (Refactor FnKind variant to hold &Fn)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 53f343f into rust-lang:master Jan 29, 2025
6 checks passed
@rustbot rustbot added this to the 1.86.0 milestone Jan 29, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Jan 29, 2025
@rust-timer
Unrolled build for rust-lang#135625
88b4dc3 
Rollup merge of rust-lang#135625 - c410-f3r:cfg-match-foo-bar-baz, r=tgross35,jhpratt

[cfg_match] Document the use of expressions.

cc rust-lang#115585

Adds documentation to this new feature introduced in rust-lang#133720.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgross35 tgross35 tgross35 left review comments

Assignees

@Noratrieb Noratrieb

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.86.0
Development

Successfully merging this pull request may close these issues.
10 participants
@c410-f3r @rustbot @rust-log-analyzer @ehuss @tgross35 @jhpratt @bors @matthiaskrgr @joboet @Noratrieb