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

Use ruff instead of pylint #308

Closed
CasperWA opened this issue Oct 21, 2024 · 0 comments · Fixed by #309
Closed

Use ruff instead of pylint #308

CasperWA opened this issue Oct 21, 2024 · 0 comments · Fixed by #309
Assignees
@CasperWA
Labels
CI/CD Continuous integration (CI) & continuous deployment (CD) enhancement New feature or request

Comments

@CasperWA
Copy link
Contributor

ruff implements the same rules as pylint as well as many more.
Furthermore, it being based on rust is much faster to run, both locally and in the CI/CD workflows - this speed increase is thought of as more of a plus for running pre-commit locally.

Finally, this will align this repository with other OTEAPI repositories.
@CasperWA CasperWA added enhancement New feature or request CI/CD Continuous integration (CI) & continuous deployment (CD) labels Oct 21, 2024
@CasperWA CasperWA self-assigned this Oct 21, 2024
@CasperWA CasperWA linked a pull request Oct 21, 2024 that will close this issue
Update dev tools #309
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CasperWA CasperWA

Labels
CI/CD Continuous integration (CI) & continuous deployment (CD) enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update dev tools EMMC-ASBL/oteapi-dlite
1 participant
@CasperWA