Skip to content

small changes to CI #54

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

Merged
merged 3 commits into from
Sep 1, 2022

Conversation

ImogenBits
Copy link
Collaborator

Split out from the docker PR, this only changes the CI files to have the flake8 config be in its own file and have CI run on PRs to this branch

@Benezivas
Copy link
Collaborator

Benezivas commented Sep 1, 2022

The proposed changes seem reasonable and can be merged without further modification.

The CI on this PR fails as the 4.0.0-rc-branch has some linting issues itself, as is now revealed. This should be fixed on that branch itself and not here.

Before merging 4.0.0-rc into main the CI should be cleaned up again.

@Benezivas Benezivas merged commit 4d04fde into Algorithmic-Battle:4.0.0-rc Sep 1, 2022
@ImogenBits ImogenBits deleted the better_linting branch September 1, 2022 21:27
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