Skip to content

pre-commit: Update isort version to 5.12.0 - #12645

Merged
raydouglass merged 1 commit into
NVIDIA:branch-23.02from
wence-:wence/fix/pre-commit-isort
Jan 30, 2023
Merged

raydouglass merged 1 commit into
NVIDIA:branch-23.02from
wence-:wence/fix/pre-commit-isort

Conversation

@wence-

@wence- wence- commented Jan 30, 2023

Copy link
Copy Markdown
Contributor

Description

poetry version 1.5.0 broke installs of isort prior to 5.11.5 (see PyCQA/isort#2077 and PyCQA/isort#2078), so we need to upgrade.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

poetry version 1.5.0 broke installs of isort prior to 5.11.5 (see
PyCQA/isort#2077 and PyCQA/isort#2078), so we need to upgrade.
@wence- wence- added tests Unit testing for project ! - Hotfix Hotfix is a bug that affects the majority of users for which there is no reasonable workaround non-breaking Non-breaking change labels Jan 30, 2023
@wence- wence- self-assigned this Jan 30, 2023
@wence- wence- added the bug Something isn't working label Jan 30, 2023
@galipremsagar galipremsagar added the 5 - Ready to Merge Testing and reviews complete, ready to merge label Jan 30, 2023
@wence-

wence- commented Jan 30, 2023

Copy link
Copy Markdown
Contributor Author

/merge

@shwina shwina removed the ! - Hotfix Hotfix is a bug that affects the majority of users for which there is no reasonable workaround label Jan 30, 2023
@shwina

shwina commented Jan 30, 2023

Copy link
Copy Markdown
Contributor

(not a hotfix, as cuDF freezes EOD today)

@msadang msadang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Revision changes looks good.

@raydouglass
raydouglass merged commit 005d821 into NVIDIA:branch-23.02 Jan 30, 2023
@wence-
wence- deleted the wence/fix/pre-commit-isort branch January 31, 2023 08:23
rapids-bot Bot pushed a commit to rapidsai/cucim that referenced this pull request Feb 1, 2023
Poetry version 1.5.0 broke installs of `isort` prior to 5.11.5 (see PyCQA/isort#2077 and PyCQA/isort#2078).

This is fixed in 5.12.0.

xref: NVIDIA/cudf#12645
rapids-bot Bot pushed a commit to rapidsai/cucim that referenced this pull request Feb 1, 2023
Poetry version 1.5.0 broke installs of `isort` prior to 5.11.5 (see PyCQA/isort#2077 and PyCQA/isort#2078).

This is fixed in 5.12.0.

xref: NVIDIA/cudf#12645

Authors:
  - AJ Schmidt (https://github.com/ajschmidt8)

Approvers:
  - https://github.com/jakirkham
  - Gregory Lee (https://github.com/grlee77)

URL: #492
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to Merge Testing and reviews complete, ready to merge bug Something isn't working non-breaking Non-breaking change tests Unit testing for project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants