Skip to content

Conversation

@ulgens
Copy link
Member

@ulgens ulgens commented Nov 3, 2025

Adds prek workflow to CI so pre-commit hooks are automatically checked.

This is currently checked by an external service that we have no control over. I also think that there are other issues with the service - a summary of those issues can be found here: j178/prek#621 (comment) This PR can also be used as a pre-commit to prek migration, for the same reasons described at django/djangoproject.com#2252

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (a895084) to head (0d5ed34).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #186   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          400       400           
  Branches        54        54           
=========================================
  Hits           400       400           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@browniebroke
Copy link
Member

I thought this would already be covered by pre-commit.ci?

@ulgens
Copy link
Member Author

ulgens commented Nov 3, 2025

@browniebroke Yep, sorry. The PR needs a better description. Updated now.

@ulgens ulgens merged commit 99426e7 into master Nov 3, 2025
12 checks passed
@ulgens ulgens deleted the prek-ci branch November 3, 2025 20:05
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.

2 participants