Skip to content

Conversation

@WeiTheShinobi
Copy link
Contributor

fixes #6158

cargo test 0 failed

first time contributed on this project🥳

@ytmimi ytmimi changed the title Fix: rustfmt should not eats attributes in inline const blogs Fix: rustfmt should not eats attributes in inline const blocks May 23, 2024
@ytmimi ytmimi changed the title Fix: rustfmt should not eats attributes in inline const blocks Fix: rustfmt should not remove inner attributes from inline const blocks May 23, 2024
Copy link
Contributor

@ytmimi ytmimi left a comment

Choose a reason for hiding this comment

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

Thanks for jumping in to work on this PR! With a few minor tweaks we should be good

@WeiTheShinobi
Copy link
Contributor Author

WeiTheShinobi commented May 24, 2024

Looks so pretty, I'm learned a lot.
Thank you!
I will do better in the next PR.

Copy link
Contributor

@ytmimi ytmimi left a comment

Choose a reason for hiding this comment

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

Thanks for making the requested changes!

@ytmimi
Copy link
Contributor

ytmimi commented Jun 1, 2024

Diff check job also passed so we're good to go 👍🏼

@ytmimi ytmimi merged commit bf7bb56 into rust-lang:master Jun 1, 2024
@ytmimi ytmimi added release-notes Needs an associated changelog entry and removed pr-ready-to-merge labels Jun 1, 2024
@WeiTheShinobi WeiTheShinobi deleted the issue_6158 branch June 10, 2024 05:46
@ytmimi ytmimi removed the release-notes Needs an associated changelog entry label Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rustfmt removes inner attributes from inline const blocks

3 participants