-
Notifications
You must be signed in to change notification settings - Fork 14
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
[#508] Use setuptools for plugin mechanism. #509
base: main
Are you sure you want to change the base?
Conversation
- Line too long error not reported when upgrading ruff tool from 0.0.278 to 0.1.1. - Not running pycodestyle, pydocstyle, isort, or flake8 during tox run. Those tools are run in other parts of the Statick testing pipeline.
…r, which are no longer supported by Statick.
This reverts commit b218a90.
…unit test with different outcomes based on version of cppcheck.
I am looking for feedback on these changes. Once the changes are agreed to be on the right track, I will open PRs for all the plugin repositories/packages. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #509 +/- ##
===========================================
+ Coverage 99.97% 100.00% +0.02%
===========================================
Files 59 59
Lines 3377 3387 +10
===========================================
+ Hits 3376 3387 +11
+ Misses 1 0 -1 ☔ View full report in Codecov by Sentry. |
We should do a new minor release before merging this change. |
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.
Overall, I like the approach, seems pretty clean. Just had some minor comments.
Good idea. That is done now. v0.10.0 released today. |
Great changes. |
Switch from yapsy to setuptools for creating plugins for discovery, reporting, and tools.