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

Adopt black in pre-commit #98

Merged
merged 3 commits into from Sep 17, 2022
Merged

Conversation

waynesun09
Copy link
Collaborator

Update using black for formatting python code, automatically apply change in the pre-commit hook.

More info about black:
https://black.readthedocs.io/en/stable/

Signed-off-by: Wayne Sun gsun@redhat.com

Update using black for formatting python code, automatically apply
change in the pre-commit hook.

More info about black:
https://black.readthedocs.io/en/stable/

Signed-off-by: Wayne Sun <gsun@redhat.com>
@waynesun09
Copy link
Collaborator Author

pre-commit have dropped 3.6 support
pre-commit/pre-commit#2215

Signed-off-by: Wayne Sun <gsun@redhat.com>
Python 3.6 will fail with pre-commit setup:

  Complete output (6 lines):
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
    File "/home/runner/.cache/pre-commit/repozbn0y53a/setup.py", line 1
      from __future__ import annotations
      ^
  SyntaxError: future feature annotations is not defined

The pre-commit project has dropped support on 3.6:

pre-commit/pre-commit#2215

Signed-off-by: Wayne Sun <gsun@redhat.com>
Copy link
Collaborator

@leelavg leelavg left a comment

Choose a reason for hiding this comment

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

Fixes: #27

@leelavg leelavg merged commit 3b156e5 into RedHatQE:main Sep 17, 2022
@waynesun09 waynesun09 deleted the pre_commit_black branch September 21, 2022 14:56
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.

None yet

2 participants