Skip to content
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

DM-45138: Update GitHub Actions configuration #207

Merged
merged 1 commit into from
Jul 12, 2024
Merged

DM-45138: Update GitHub Actions configuration #207

merged 1 commit into from
Jul 12, 2024

Conversation

rra
Copy link
Member

@rra rra commented Jul 12, 2024

Update the GitHub Actions configuration to match the current fastapi_safir_app template (with some changes that are still pending review). Stop testing with a matrix of Python versions, since services need only support one version. Stop using neophile and instead run make update-deps directly. Use the new tox requirements file instead of manually listing plugins. Use an environment variable to hold the Python version so that it can be changed in fewer places. Fix the way errors from the periodic CI job are reported to avoid a discouraged GitHub Actions construction.

Update the GitHub Actions configuration to match the current
fastapi_safir_app template (with some changes that are still pending
review). Stop testing with a matrix of Python versions, since services
need only support one version. Stop using neophile and instead run
make update-deps directly. Use the new tox requirements file instead
of manually listing plugins. Use an environment variable to hold the
Python version so that it can be changed in fewer places. Fix the way
errors from the periodic CI job are reported to avoid a discouraged
GitHub Actions construction.
@rra rra merged commit 3f6fbea into main Jul 12, 2024
3 checks passed
@rra rra deleted the tickets/DM-45138 branch July 12, 2024 18:12
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.

1 participant