-
Notifications
You must be signed in to change notification settings - Fork 13.6k
set default for build.tidy-extra-checks to auto run all but shellcheck #144461
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
Conversation
…t shellcheck shellcheck is the only one of these tests which is not run in CI.
This PR modifies If appropriate, please update |
Can you measure how long it takes on your PC with/without the |
The job Click to see the possible cause of the failure (guessed by this bot)
|
I believe it adds around 0.1 seconds, tho I have another PR i'm working on that lowers that down to 0.05 by eliminating the repeated calls to git.
I was planning on adding a change tracker entry, but how should I document it in boostrap.example.toml? say it can be permanently disabled by setting it to the empty string? mention that it should be disabled on hdd systems with less memory? |
also, it looks like this should be disabled on CI unless explicitly enabled? otherwise every dist job will enable this, and not all of them have all the extra tools needed. other option is making all those jobs install those tools. also i think this would be run by distro maintainers where it previously was not. perhaps a better idea would be enabling it by default only in the |
Well we should definitely document the default there, and also the way how to disable it with the empty string, yes.
Dist jobs shouldn't ever run tidy. That being said, the default should indeed be disabled on CI, because a lot of test jobs do run just |
…extra-checks-enable-for-tools, r=Kobzol bootstrap: enable tidy auto extra checks on tools profile alternative to rust-lang#144461 this won't affect CI or any `./configure` based workflows, and will also not affect every rust contributor like that PR will. a slower rollout of this feature should reduce disruption if issues are discovered with it. r? `@Kobzol`
…extra-checks-enable-for-tools, r=Kobzol bootstrap: enable tidy auto extra checks on tools profile alternative to rust-lang#144461 this won't affect CI or any `./configure` based workflows, and will also not affect every rust contributor like that PR will. a slower rollout of this feature should reduce disruption if issues are discovered with it. r? ``@Kobzol``
Rollup merge of #144599 - lolbinarycat:bootstrap-build.tidy-extra-checks-enable-for-tools, r=Kobzol bootstrap: enable tidy auto extra checks on tools profile alternative to #144461 this won't affect CI or any `./configure` based workflows, and will also not affect every rust contributor like that PR will. a slower rollout of this feature should reduce disruption if issues are discovered with it. r? ``@Kobzol``
☔ The latest upstream changes (presumably #144692) made this pull request unmergeable. Please resolve the merge conflicts. |
shellcheck is the only one of these tests which is not run in CI.
the
auto:
system seems to be working decently well, so I think it makes sense to enable it by default at some point.unsure if we want to let this sit for a bit to make sure no issues come up, or optimize extra checks more, or if it's good to just move ahead with this. my only concern would be if it makes tidy extra slow on hdd systems.
r? @Kobzol