ci: Add flagsmith-lint-tests pre-commit hook#6933
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6933 +/- ##
=======================================
Coverage 98.31% 98.31%
=======================================
Files 1335 1335
Lines 49784 49784
=======================================
Hits 48947 48947
Misses 837 837 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
32b2092 to
d3c94e4
Compare
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
|
Add the flagsmith-lint-tests pre-commit hook from flagsmith-common (fv3.4.0) and add # noqa comments to all existing test violations.
d3c94e4 to
a5c3377
Compare
docs/if required so people know about the feature.Changes
flagsmith-lint-testspre-commit hook from flagsmith-common (fv3.4.0)# noqacomments to all existing violations (2052 FT003, 341 FT004)How did you test this code?
pre-commit run flagsmith-lint-tests --all-filespasses.