-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
internal: refactor prefer_no_std
/prefer_prelude
bools into a struct
#17252
internal: refactor prefer_no_std
/prefer_prelude
bools into a struct
#17252
Conversation
4da6dbf
to
04d9910
Compare
04d9910
to
dc565d4
Compare
Rebased it given my PR had a bunch of conflicts touching the same parts :) The new |
…to-struct, r=Veykril internal: refactor `prefer_no_std`/`prefer_prelude` bools into a struct I noticed that there's a large number of functions/arguments during an unrelated change that take two booleans and realized they're _probably_ better off being in a single struct—less error-prone, etc. Feel free to suggest a better name than `ImportPathConfig`/close this entirely! I can also make these args enums; just hopefully making this a little more misuse-resistant.
💔 Test failed - checks-actions |
dc565d4
to
3eb607c
Compare
small rebase mishap |
…to-struct, r=Veykril internal: refactor `prefer_no_std`/`prefer_prelude` bools into a struct I noticed that there's a large number of functions/arguments during an unrelated change that take two booleans and realized they're _probably_ better off being in a single struct—less error-prone, etc. Feel free to suggest a better name than `ImportPathConfig`/close this entirely! I can also make these args enums; just hopefully making this a little more misuse-resistant.
💔 Test failed - checks-actions |
3eb607c
to
3a7dcf9
Compare
@bors r+ |
☀️ Test successful - checks-actions |
I noticed that there's a large number of functions/arguments during an unrelated change that take two booleans and realized they're probably better off being in a single struct—less error-prone, etc.
Feel free to suggest a better name than
ImportPathConfig
/close this entirely! I can also make these args enums; just hopefully making this a little more misuse-resistant.