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

style: Replace black . with ruff format for improved user experience to match grass repo #1298

Merged
merged 5 commits into from
Feb 4, 2025

Conversation

echoix
Copy link
Member

@echoix echoix commented Feb 4, 2025

Equivalent PR of OSGeo/grass#5023.

Applied here to help keep both repos contributing workflows similar.

For the post-pr-reviews workflow, it is intended to keep the old name of the workflow, so it can still continue to send review comments for PRs that aren't updated yet. The old workflow name could be removed from the list of triggers later on (like once all currently open PRs are either closed, merged, or updated from main. That file now only has the list of workflow names to run against that differs from the file in the main repo (that's why we see a bit more changes than the strict minimum).
I intend to group other tools in that workflow, thus the generic tile and file name, but for now, flake8 (running in its own workflow file) is a required check, so the name of the job+workflow cannot be changed unless the repo's settings are changed.

@echoix echoix merged commit e046eb4 into OSGeo:grass8 Feb 4, 2025
7 checks passed
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.

1 participant