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.

Sign up for GitHub

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

Jump to bottom

ci: switch to ruff #795

Merged
merged 6 commits into from
Jun 17, 2024
Merged

ci: switch to ruff #795

merged 6 commits into from
Jun 17, 2024

Conversation

lukapeschke
Copy link
Contributor

No description provided.

@lukapeschke
feat(deps): replace flake8 + isort + black with ruff
0973feb 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
@lukapeschke
refactor: adapt tooling config
94739e6 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
@lukapeschke
refactor: adapt Makefile commands
d43c955 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
@lukapeschke
chore: make format
722c07a 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
@lukapeschke
refactor: adapt pre-commit config
53b52a7 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
@lukapeschke lukapeschke added enhancement New feature or request dependencies Pull requests that update a dependency file labels Jun 17, 2024
@lukapeschke lukapeschke self-assigned this Jun 17, 2024
@lukapeschke
fix: use poetry run in Makefile
3b53b94 
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
julien-pinchelimouroux
julien-pinchelimouroux approved these changes Jun 17, 2024
View reviewed changes
Copy link

@julien-pinchelimouroux julien-pinchelimouroux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour les confs spécifiques (par ex max-line-length = 100 ici) on en n'a pas non plus sur les autres repos qui sont déjà sous ruff j'imagine ?

@lukapeschke lukapeschke merged commit b63f861 into main Jun 17, 2024
7 checks passed
@lukapeschke lukapeschke deleted the switch-to-ruff branch June 17, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julien-pinchelimouroux julien-pinchelimouroux julien-pinchelimouroux approved these changes

Assignees

@lukapeschke lukapeschke

Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lukapeschke @julien-pinchelimouroux