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

GitHub actions #340

Merged
merged 21 commits into from
Dec 28, 2021
Merged

GitHub actions #340

merged 21 commits into from
Dec 28, 2021

Conversation

carlio
Copy link
Collaborator

@carlio carlio commented Dec 28, 2021

I was fixing up tests (see #338) and was going to move to github-actions then saw #336 . However it now has a bunch of conflicts with master due my test fixes so I will cherry-pick things from it into this.

carlio and others added 19 commits December 28, 2021 10:13
… outputs, so adding these in. Also, fixing the consider-using-f-string warnings from tests (#338)
DjangoInstalledChecker will catch the condition and inform the user,
but we should not crash before that checker gets executed.
mesage ID seems to have changed
run tests for only 1 Python + Django combination
because converting to lcov format simply doesn't work
… call tox env py36-django2 instead of py36-django20 - so, quote the .0 django versions to prevent YAML casting...
@carlio carlio marked this pull request as ready for review December 28, 2021 14:06
@carlio carlio merged commit adfc253 into master Dec 28, 2021
@carlio carlio deleted the github-actions branch December 28, 2021 14:34
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