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

[CI] Bump black version to 22.3.0 #10960

Merged
merged 1 commit into from
Apr 11, 2022
Merged

[CI] Bump black version to 22.3.0 #10960

merged 1 commit into from
Apr 11, 2022

Conversation

leandron
Copy link
Contributor

  • Make all required adjusts in the code to comply with the new version
  • The major change in black seems to be that x ** y now needs to be formatted as x**y in Python
  • Update ci-lint to v0.71, based on tlcpackstaging/ci_lint:20220411-060305-45f3d4a52

cc @areusch @driazati @Mousius @manupa-arm @Leo-arm

Copy link
Contributor

@NicolaLancellotti NicolaLancellotti left a comment

Choose a reason for hiding this comment

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

LGTM!

@leo-blonk
Copy link
Contributor

LGTM2!

* Make all required adjusts in the code to comply with the new version
* Upadte ci-lint to v0.71, based on tlcpackstaging/ci_lint:20220411-060305-45f3d4a52
@areusch
Copy link
Contributor

areusch commented Apr 11, 2022

cc @mehrdadh

@areusch areusch merged commit 89061fa into main Apr 11, 2022
Lucien0 pushed a commit to Lucien0/tvm that referenced this pull request Apr 19, 2022
* Make all required adjusts in the code to comply with the new version
* Upadte ci-lint to v0.71, based on tlcpackstaging/ci_lint:20220411-060305-45f3d4a52
altanh pushed a commit to altanh/tvm that referenced this pull request Apr 28, 2022
* Make all required adjusts in the code to comply with the new version
* Upadte ci-lint to v0.71, based on tlcpackstaging/ci_lint:20220411-060305-45f3d4a52
@tqchen tqchen deleted the update-ci-lint-v071 branch May 5, 2022 22:50
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.

5 participants