Skip to content

MNT Add pre-commit setup - #269

Merged
lesteve merged 5 commits into
joblib:masterfrom
lesteve:add-precommit
Oct 18, 2026
Merged

MNT Add pre-commit setup#269
lesteve merged 5 commits into
joblib:masterfrom
lesteve:add-precommit

Conversation

@lesteve

@lesteve lesteve commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • run pre-commit in linting step (as in joblib)
  • took the pre-commit config from joblib (updated the ruff version)
  • ran pre-commit run --all-files locally to fix the issues

Tested CI lint failure output in build log

@lesteve

lesteve commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

Failure build log is #261.

@lesteve lesteve added this to Labs Oct 18, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Labs Oct 18, 2026
@lesteve
lesteve merged commit 7302154 into joblib:master Oct 18, 2026
25 of 26 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Labs Oct 18, 2026
@lesteve
lesteve deleted the add-precommit branch October 18, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lesteve