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
Enhance and optimize PR GitHub workflows #1365
Conversation
This will make PR checks faster and more comprehensive, but you'll need to update "required status checks" in repo settings. @timgl I think all besides Cypress should be required. |
requirements.txt
Outdated
@@ -23,6 +23,7 @@ django-redis==4.12.1 | |||
django-statsd==2.5.2 | |||
djangorestframework==3.11.0 | |||
djangorestframework-csv==2.1.0 | |||
freezegun==0.3.15 |
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.
don't thinik that's supposed to be in here?
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.
Hm, wouldn't be a problem but yeah, since only tests need freezegun and they're run with prod requirements.txt only, I just put it into the GH workflow.
This reverts commit 460e3942d10194e9f46c8a2196ebd9ecc1f23c6f.
22e92f9
to
d1cebd5
Compare
Changes
This improves GitHub workflow naming and separation, uses parallel jobs, updates ESLint linting, and adds prettier plus black checks. Also makes use of GitHub check annotations.