-
Notifications
You must be signed in to change notification settings - Fork 968
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
New linter for test(symbol) #6120
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @MichaelChirico and the rest of your teammates on |
great but do we need to turn off the linter in |
it only happens once now in tests.Rraw (hence the nolint). It happens a few more times in froll, hence the config entry in #6121. I'd rather not touch those tests until the existing froll PRs are eventually cleared. |
great |
2832585
to
99b0b23
Compare
00b1406
to
1d045f8
Compare
Merge activity
|
99b0b23
to
dc2d6ed
Compare
1d045f8
to
ed0d93d
Compare
This is a follow-up to #6041 that will prevent regression on that front going forward.