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

Add a GitHub Actions CI workflow #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

bmillwood
Copy link

@bmillwood bmillwood commented May 16, 2024

I used the template that GitHub gives you from clicking the Actions button, plus an update because the template expected a different requirements.txt filename.

I expect the pytest step to be fairly uncontroversial, but I'm less sure about the flake8 step. Happy to drop it if you don't like the recommendations. If you do like the recommendations but you don't have time to actually fix them, I may have a go at doing it myself (or else we could just still put it in but ignore it / disable it for now).

The action seems not to be running on this PR, so see bmillwood#1 for what it looks like to run it.

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.

None yet

1 participant