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 Action to lint Python code #455

Merged
merged 5 commits into from
Mar 17, 2022
Merged

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Feb 15, 2022

GHA may be a bit easier to work with than AppVeyor

Test results: https://github.com/cclauss/multibuild/actions

@multibuilder
Copy link
Member

I created a test.yml action as a base for running tests. I don't think we want to begin with a linting task, that seems a bit advanced for now.

@cclauss
Copy link
Contributor Author

cclauss commented Feb 15, 2022

It seems a bit advanced.

The tests pass. ;-) https://github.com/cclauss/multibuild/actions

@mattip
Copy link
Collaborator

mattip commented Feb 15, 2022

@matthew-brett any thoughts about adding linting?

@cclauss
Copy link
Contributor Author

cclauss commented Mar 17, 2022

Status?

@mattip
Copy link
Collaborator

mattip commented Mar 17, 2022

The CI run seems very fast, let's put this in. If it gets too noisy we can adjust it.

@mattip mattip merged commit e7aa6ee into multi-build:devel Mar 17, 2022
@cclauss cclauss deleted the patch-2 branch March 17, 2022 08:13
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

3 participants