-
Notifications
You must be signed in to change notification settings - Fork 29
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 python linting in CI #79
base: master
Are you sure you want to change the base?
Conversation
We were kinda talking about maybe moving to ruff8 so I'm not sure this is the right direction '-' Also I'm more in favor of automatic tooling doing the job in side than making merging PR harder since most of our contributors aren't professional developers in a professional context, I don't think we want to make contributing harder than it's already is :/ |
You’re right, I’ll use ruff instead. For your second point, i’m not sure i understand : this action is only checking, not committing or creating merge requests. |
…hen commit, only on push on master
@Psycojoker I reorganized the github actions in two:
|
Update actions/checkout and actions/setup-python
No description provided.