Skip to content

only dump defaults for stable configs when running stable rustfmt#5597

Open
ytmimi wants to merge 1 commit intorust-lang:mainfrom
ytmimi:issue_5594
Open

only dump defaults for stable configs when running stable rustfmt#5597
ytmimi wants to merge 1 commit intorust-lang:mainfrom
ytmimi:issue_5594

Conversation

@ytmimi
Copy link
Contributor

@ytmimi ytmimi commented Nov 7, 2022

Fixes #5594

Previously running rustfmt with --print-config default would always return a list of all config options and their default values.

Now, when running the command on stable rustfmt, we only return the default options for configs that have been marked as stable.

Previously running rustfmt with `--print-config default` would always
return a list of all config options and their default values.

Now, when running the command on stable rustfmt we only return the
default options for configs that have been marked as stable.
@jieyouxu jieyouxu added S-waiting-on-review Status: awaiting review from the assignee but also interested parties. and removed pr-not-reviewed labels Feb 23, 2026
@jieyouxu jieyouxu closed this Feb 27, 2026
@rustbot rustbot removed the S-waiting-on-review Status: awaiting review from the assignee but also interested parties. label Feb 27, 2026
@jieyouxu jieyouxu reopened this Feb 27, 2026
@rustbot rustbot added the S-waiting-on-review Status: awaiting review from the assignee but also interested parties. label Feb 27, 2026
Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

The general notion looks good to me, but I suspect this might need to be updated since the PR is open since forever :D

@rustbot author

View changes since this review

@jieyouxu jieyouxu self-assigned this Feb 27, 2026
@rustbot rustbot added S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: awaiting review from the assignee but also interested parties. labels Feb 27, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 27, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generating config file includes all unstable features and generates bunch of warnings

3 participants