Move lints configuration to manifest#822
Conversation
|
|
||
| [lints.clippy] | ||
| use_self = "warn" | ||
| too_many_arguments = "allow" |
There was a problem hiding this comment.
Unfortunately it is not possible to override lints from the root.
So here the common lints are duplicated to be able to add the three "allow"
MarijnS95
left a comment
There was a problem hiding this comment.
I am intentionally not pushing this to the ash repo because:
- Lints are explicitly inherited from the workspace, not implicitly forced down;
- Unnecessary MSRV bump;
- Strange duplication in overrides.
Can I close this as "Not planned"?
No problems... |
|
I also found it strange that workspace lints can't be overridden at the workspace member level. |
|
I dropped the MSRV bump. What about the lint fixes and the "resolve" warning fix ? Should we keep them ? |
|
I moved the PS: I don't expect this to be merged. Just showcasing. |
e33abe0 to
0bb826e
Compare
Note that everyone gets notifications when you're debugging by means of (force-)pushing. |
Sorry about that. I'll be more careful next time. |
|
Current status / limitations:
|
This is possible since Rust 1.74.0.
This PR shows what it looks like.