Skip to content
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

[#422] tool plugin refactor #423

Merged
merged 9 commits into from
Jun 14, 2022
Merged

Conversation

xydesa
Copy link
Collaborator

@xydesa xydesa commented Jun 8, 2022

#422
Restructure tool plugins to not run if no files of their desired type are available. In addition, I pulled out as much common code as made sense and converted as many plugins to the new structure as possible.

@xydesa xydesa changed the title 422 tool plugin refactor #422 tool plugin refactor Jun 8, 2022
@xydesa xydesa changed the title #422 tool plugin refactor [#422] tool plugin refactor Jun 8, 2022
@codecov
Copy link

codecov bot commented Jun 8, 2022

Codecov Report

Merging #423 (e10af40) into main (5fef517) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #423   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           56        56           
  Lines         3319      3257   -62     
=========================================
- Hits          3319      3257   -62     
Impacted Files Coverage Δ
statick_tool/plugins/tool/bandit_tool_plugin.py 100.00% <100.00%> (ø)
statick_tool/plugins/tool/black_tool_plugin.py 100.00% <100.00%> (ø)
...atick_tool/plugins/tool/catkin_lint_tool_plugin.py 100.00% <100.00%> (ø)
statick_tool/plugins/tool/cccc_tool_plugin.py 100.00% <100.00%> (ø)
...tick_tool/plugins/tool/clang_format_tool_plugin.py 100.00% <100.00%> (ø)
...tatick_tool/plugins/tool/clang_tidy_tool_plugin.py 100.00% <100.00%> (ø)
statick_tool/plugins/tool/cmakelint_tool_plugin.py 100.00% <100.00%> (ø)
statick_tool/plugins/tool/cppcheck_tool_plugin.py 100.00% <100.00%> (ø)
statick_tool/plugins/tool/cpplint_tool_plugin.py 100.00% <100.00%> (ø)
...tick_tool/plugins/tool/docformatter_tool_plugin.py 100.00% <100.00%> (ø)
... and 17 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@gregtkogut gregtkogut merged commit dd0f6a4 into sscpac:main Jun 14, 2022
denewiler pushed a commit to sscpac/statick-md that referenced this pull request Oct 10, 2022
…inning pycodestyle version to match flake8 version. (#60)
denewiler pushed a commit to sscpac/statick-tooling that referenced this pull request Oct 10, 2022
…n version 5. Update to tool plugins to match new structure [sscpac/statick#423].  (#25)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants