Add two more `pre-commit` hooks
https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#detect-aws-credentials
https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#forbid-submodules
Description
This PR adds two more checks or tests to our pre-commit framework.
The two hooks are from the official pre-commit organization.
No changes to the codebase but having extra checks stops regressions.
Types of changes
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Enhancement (improves an existing feature and functionality)
- [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
- [X] build/CI
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
- [ ] Major
- [ ] Minor
Bug Severity
- [ ] BLOCKER
- [ ] Critical
- [ ] Major
- [ ] Minor
- [ ] Trivial
Screenshots (if appropriate):
How Has This Been Tested?
Ran pre-commit run --all-files
How did you try to break this feature and the system with this change?
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 15.28%. Comparing base (
21af134) to head (8523bb7).
Additional details and impacted files
@@ Coverage Diff @@
## main #9077 +/- ##
===========================================
Coverage 15.28% 15.28%
Complexity 11535 11535
===========================================
Files 5425 5425
Lines 474138 474138
Branches 58984 60911 +1927
===========================================
+ Hits 72486 72487 +1
Misses 393584 393584
+ Partials 8068 8067 -1
| Flag | Coverage Δ | |
|---|---|---|
| uitests | 4.26% <ø> (ø) |
|
| unittests | 16.02% <ø> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@blueorangutan package
@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9602