-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add workflow to run checks for the API and site #64
Conversation
- Include Python dev dependencies for checks runner - Configure Python linting tools and provide compatible VS Code settings - Update production deployment to run only when checks succeed
Deploy preview for irvinehacks-site-2024 ready!
|
Deploy preview for irvinehacks-site-2024-sanity ready!
|
Update: the relevant SAML is not spotted during pull request since those lines were not changed but would be caught if we merged and pushed to |
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.
Looking good, thanks for working on this. One small question:
Sounds good, please squash merge. |
89e6135
to
ceb82ff
Compare
As we did before with last year's HackUCI-Site repository, we can use a GitHub Actions workflow to run checks on pull requests and pushes to
main
.The checks run for this pull request will initially fail because there is a type error in
saml.py
which will demonstrate that this runner works.Changes