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

[MNT] Update pre commit hooks and run all #1630

Merged
merged 2 commits into from
Aug 30, 2024
Merged

[MNT] Update pre commit hooks and run all #1630

merged 2 commits into from
Aug 30, 2024

Conversation

yarnabrina
Copy link
Member

  1. updates all the pre-commit hooks to their latest versions using pre-commit autoupdate
  2. runs all hooks on all files using pre-commit run --all-files

fkiraly
fkiraly approved these changes Aug 30, 2024
View reviewed changes
Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

Seems like it did not change too many files?

@fkiraly fkiraly changed the title Update pre commit hooks and run all [MNT] Update pre commit hooks and run all Aug 30, 2024
@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label Aug 30, 2024
@fkiraly fkiraly merged commit 752d099 into sktime:master Aug 30, 2024
15 checks passed
@yarnabrina yarnabrina deleted the update-pre-commit-hooks-and-run-all branch August 30, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fkiraly fkiraly fkiraly approved these changes

@jdb78 jdb78 Awaiting requested review from jdb78

Assignees
No one assigned
Labels
maintenance Continuous integration, unit testing & package distribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yarnabrina @fkiraly