Change uninlined_format_args into a style lint

[nyurik]

As previously discussed, the uninlined_format_args should probably be a part of the default style because println!("{}", foo) is not as concise or easy to understand as println!("{foo}")

changelog: [uninlined_format_args]: change to be the default style

[rust-highfive]

r? @Alexendoo

(rust-highfive has picked a reviewer for you, use r? to override)

[llogiq]

Taking over review from @Alexendoo to reduce his load.

I'd like to see an //edition:2018 test. I personally think the lint should probably not apply there, right?

r? @llogiq

[nyurik]

I'd like to see an //edition:2018 test. I personally think the lint should probably not apply there, right?

@llogiq take a look at #9605 -- I tried to add a 2018 edition, and fix the bug with the panic, but somehow the test does not run as 2018th edition - could you see if i need to do something else there to fix it? Thanks!

[llogiq]

Ok, so I'll try getting #9605 merged first.

[nyurik]

This one is ready to go

[llogiq]

@bors r+

[bors]

📌 Commit 4936848 has been approved by llogiq

It is now in the queue for this repository.

[bors]

⌛ Testing commit 4936848 with merge 58ef56e...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 58ef56e to master...