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

[MNT] Bump isort to 5.12.0 in pre-commit config #4167

Merged
merged 5 commits into from
Jan 28, 2023

Conversation

snnbotchway
Copy link
Contributor

I updated the sort version in pre-commit to stop pre-commit checks from failing.

Closes #4162

@fkiraly
Copy link
Collaborator

fkiraly commented Jan 28, 2023

Thanks a lot for helping out with this, @snnbotchway!

I thought I already tried updating isort - see here for the things I tried:
#4163

But I see you did it differently, so let's try what this does.

@snnbotchway
Copy link
Contributor Author

A new release(5.12.0) was just released an hour ago.
This version solves the issue.

@snnbotchway snnbotchway changed the title [MNT] Update isort [MNT] Bump isort to 5.12.0 in pre-commit config Jan 28, 2023
@fkiraly
Copy link
Collaborator

fkiraly commented Jan 28, 2023

Ah! Well spotted!

@snnbotchway
Copy link
Contributor Author

Thanks a lot for reviewing @fkiraly
Merging this will be my first contribution

@fkiraly
Copy link
Collaborator

fkiraly commented Jan 28, 2023

yes, that's true!
Feel free to add yourself to all-contributorsrc with a maintenance contribution badge!

@snnbotchway
Copy link
Contributor Author

Sure, done!

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.

Well, this works - thanks!

@fkiraly fkiraly merged commit bee4c01 into sktime:main Jan 28, 2023
@fkiraly
Copy link
Collaborator

fkiraly commented Jan 28, 2023

Thanks for fixing, @snnbotchway! This has caused me quite a headache today!

@snnbotchway
Copy link
Contributor Author

snnbotchway commented Jan 28, 2023 via email

@fkiraly
Copy link
Collaborator

fkiraly commented Jan 28, 2023

always!
If you'd like to continue contributing, free to pick up one of the good first issues and/or join one of the community sessions. Much appreciated.

@snnbotchway
Copy link
Contributor Author

snnbotchway commented Jan 28, 2023 via email

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.

[BUG] CI fails in code_quality isort hook, environment creation, at preparing metadata via pyproject.toml
2 participants