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

Commits on Sep 15, 2022

  1. Adopt black in pre-commit

    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 committed Sep 15, 2022
    Configuration menu
    Copy the full SHA
    87fb12a View commit details
    Browse the repository at this point in the history

Commits on Sep 16, 2022

  1. Update github action versions to latest

    Signed-off-by: Wayne Sun <gsun@redhat.com>
    waynesun09 committed Sep 16, 2022
    Configuration menu
    Copy the full SHA
    682d324 View commit details
    Browse the repository at this point in the history
  2. Skip python 3.6 for pre-commit

    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>
    waynesun09 committed Sep 16, 2022
    Configuration menu
    Copy the full SHA
    f56d8f2 View commit details
    Browse the repository at this point in the history