search

rust-lang / rustfmt

Format Rust code
https://rust-lang.github.io/rustfmt/
Apache License 2.0
6.05k stars 888 forks source link

refactor - show file path in error message when parsing config from toml #6323

Closed ytmimi closed 2 months ago

ytmimi commented 2 months ago

Fixes #6302

Display the path to the toml config file that rustfmt failed to parse configurations from.

ytmimi commented 2 months ago

@rufevean opening up this PR instead of https://github.com/rust-lang/rustfmt/pull/6310. Not sure what the issue was but this contains all your work boiled down into a single commit. Thanks for you help!

rufevean commented 2 months ago

@rufevean opening up this PR instead of #6310. Not sure what the issue was but this contains all your work boiled down into a single commit. Thanks for you help!

thats awesome, thanks a lot for your guidance. also just curious, will i be one of the contributors now? or should i have to do more ?

ytmimi commented 2 months ago

@rufevean I think we're all set! Just rerunning the failed integration job right now, but assuming that it passes I'll get this merged

rufevean commented 2 months ago

Awesome,thank you