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 Actions: Pin ruff to the same version as pre-commit #396

Merged
merged 5 commits into from
Apr 20, 2024

Conversation

arichardson
Copy link
Member

Similar to #395
but centralizing it in requirements.txt

Similar to #395
but centralizing it in requirements.txt
ruff does not support python 3.6 anymore (and we should probably drop it
here as well), so only check that the tests still pass with the minimum
python version rather than also running the linters. Any problems they
find should be discovered by the other jobs.
This script should only run tests to check that cheribuild still works
rather than also running the linting. This made sense when we didn't have
pre-commit enabled, but now pre-commit takes care of that.
Placing it in requirements.txt means the docker images try to install it
even though they don't need it.
@arichardson arichardson force-pushed the pin-ruff-alternative branch from ef306fe to 0474197 Compare April 20, 2024 20:26
@arichardson arichardson merged commit a398046 into main Apr 20, 2024
5 checks passed
@arichardson arichardson deleted the pin-ruff-alternative branch April 20, 2024 20:44
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