-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(HIS): introduction to the HIS checker #52
Conversation
Signed-off-by: Afonso Santos <afomms@gmail.com>
Signed-off-by: Afonso Santos <afomms@gmail.com>
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.
Just tried with bao-partitioner. LGTM, however you still need to commit the his_checker.py
with x
permissions.
Sure! Just changed the permissions in the last commit |
@AfonsoSantos96 can you open a draft PR for the wip/his-metrics branch, so that you can discuss the overall state of the metrics there? |
Signed-off-by: Afonso Santos <afomms@gmail.com>
Signed-off-by: Afonso Santos <afomms@gmail.com>
941ca01
to
fc92083
Compare
This is the introduction PR to add the HIS metric checker to the bao-ci workflow.
The HIS metrics define recommended thresholds for a set of key code quality metrics (explicit in afterward PRs).
This PR still does not include the checker script itself but prepares the CI structure to eventually run it.
Type of change
Checklist: