-
Notifications
You must be signed in to change notification settings - Fork 16
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
Run SonarSource on demand. #1478
Conversation
16e8256
to
43cdf49
Compare
43cdf49
to
454bf6a
Compare
454bf6a
to
a098ed9
Compare
a098ed9
to
efbf9cf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Mentioning this in CONTRIBUTING.rst
might be helpful?
BTW, it's now faster than the Windows build; and about as fast as the documentation CI. Might be worth just running it to ensure it doesn't rot away. |
Let's leave it in place for now. When it's the last thing to go green, we can turn it off. |
No description provided.