Skip to content

style: Clarify nullary call and () no-break rule applies past max width#156826

Open
ajasad25 wants to merge 1 commit into
rust-lang:mainfrom
ajasad25:style-never-break-empty-parens-152761
Open

style: Clarify nullary call and () no-break rule applies past max width#156826
ajasad25 wants to merge 1 commit into
rust-lang:mainfrom
ajasad25:style-never-break-empty-parens-152761

Conversation

@ajasad25
Copy link
Copy Markdown

…idth

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 22, 2026

Some changes occurred in src/doc/style-guide

cc @rust-lang/style

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-style Relevant to the style team, which will review and decide on the PR/issue. labels May 22, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 22, 2026

r? @traviscross

rustbot has assigned @traviscross.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: style-team
  • style-team expanded to calebcartwright, joshtriplett, traviscross
  • Random selection from calebcartwright, traviscross

@ajasad25
Copy link
Copy Markdown
Author

ajasad25 commented May 28, 2026

Fixes #152761

@ajasad25
Copy link
Copy Markdown
Author

@traviscross please take a look.

@joshtriplett joshtriplett changed the title style: Clarify nullary call and () no-break rule applies past max w… style: Clarify nullary call and () no-break rule applies past max width May 30, 2026
@joshtriplett
Copy link
Copy Markdown
Member

This looks like it implements exactly what was FCPed. Thank you!

@bors r+ rollup

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 30, 2026

📌 Commit 0ef4e11 has been approved by joshtriplett

It is now in the queue for this repository.

🌲 The tree is currently closed for pull requests below priority 1. This pull request will be tested once the tree is reopened.

@rust-bors rust-bors Bot 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 May 30, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…arens-152761, r=joshtriplett

style: Clarify nullary call and `()` no-break rule applies past max width

…idth
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…arens-152761, r=joshtriplett

style: Clarify nullary call and `()` no-break rule applies past max width

…idth
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…arens-152761, r=joshtriplett

style: Clarify nullary call and `()` no-break rule applies past max width

…idth
rust-bors Bot pushed a commit that referenced this pull request May 30, 2026
…uwer

Rollup of 12 pull requests

Successful merges:

 - #154591 (Remove `will_cache_on_disk_for_key_fn`)
 - #156672 (Misc improvements to coroutine transform code)
 - #157027 (HIR ty lowering: Move some things into submodules)
 - #157051 (Allow two object files for a single CGU in CompiledModule)
 - #157100 (Some more per owner things)
 - #153497 (Use `trait_object_dummy_self` more & heavily fix+update related docs)
 - #155638 (Fix tupled closure signature in `AsyncFn` arg mismatch diagnostic)
 - #156826 (style: Clarify nullary call and `()` no-break rule applies past max width)
 - #157004 (Remove unused functions in `value_analysis.rs`)
 - #157032 (Fixed more &x ->&mut x suggestions)
 - #157033 (Note irrefutable while let in loop type errors)
 - #157139 (compiler: `ops::RangeInclusive` → `range::RangeInclusive`)

Failed merges:

 - #156875 (Correct and document semantics of `yield` terminator)
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. T-style Relevant to the style team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants