-
Notifications
You must be signed in to change notification settings - Fork 1k
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
make the 440 version linter an error #15962
make the 440 version linter an error #15962
Conversation
The tool loads - this should be a warning as I originally defined them. We keep getting bit by the tools-iuc not blocking PRs on warning but that isn't a reason to change warnings to errors. |
Thanks for the feedback. fine for me :)
could you point me to the docs?
OK. I will open a PR changing the workflow accordingly. |
It does load, but it's in the wrong place in the tool lineage, which I'd say it's an error not a warning. |
Then lets either add this PR or tighten the definition and doc of what an error / warning is |
IUC fails now on warn. So I'm happy. 😎 |
xref #15945 (comment)
How to test the changes?
(Select all options that apply)
License