You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Pass -A unused for the rustfix-succeeded check, too; or
Stop passing -A unused for UI tests.
The present author gave an argument for this in reconsider magic -A unused for UI tests? #43896, but the reception seemed unfavorable (one 👎 react and one negatively-disposed comment). If it turned out that there was support for this after all, we would want someone to edit existing tests to not produce spurious unused warnings, which would be a very large PR.
UI tests run with
-A unused, but UI tests with therun-rustfixdirective go on to apply suggestions and then check that there are no warnings, without passing any lint flags. The inconsistency is a papercut–footgun for UI test authors. We should do exactly one of—-A unusedfor the rustfix-succeeded check, too; or-A unusedfor UI tests.-A unusedfor UI tests? #43896, but the reception seemed unfavorable (one 👎 react and one negatively-disposed comment). If it turned out that there was support for this after all, we would want someone to edit existing tests to not produce spuriousunusedwarnings, which would be a very large PR.