Skip to content

Commit

Permalink
ext: Stop excluding 'ext/testlib' from pre-commit and format (#267)
Browse files Browse the repository at this point in the history
Though in "ext" this directory is regularly modified. `pre-commit`
should run on these files.

This PR includes running `pre-commit run --files ext/testlib` to
reformat the files in "ext/testlib" using Python Black.
  • Loading branch information
BobbyRBruce authored Sep 6, 2023
2 parents cc757cf + 96144f9 commit e80cde0
Show file tree
Hide file tree
Showing 19 changed files with 962 additions and 716 deletions.
3 changes: 3 additions & 0 deletions .git-blame-ignore-revs
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,6 @@ c3bd8eb1214cbebbc92c7958b80aa06913bce3ba

# A commit which ran flynt all Python files.
e73655d038cdfa68964109044e33c9a6e7d85ac9

# A commit which ran pre-commit on ext/testlib
9e1afdecefaf910fa6e266f29dc480a32b0fa83e
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ default_language_version:

exclude: |
(?x)^(
ext/.*|
ext/(?!testlib/).*|
build/.*|
src/systemc/ext/.*|
src/systemc/tests/.*/.*|
Expand Down
6 changes: 3 additions & 3 deletions ext/testlib/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@
from .configuration import *
from .main import main

#TODO Remove this awkward bootstrap
#FIXME
# TODO Remove this awkward bootstrap
# FIXME
from gem5 import *

#TODO Remove this as an export, users should getcwd from os
# TODO Remove this as an export, users should getcwd from os
from os import getcwd
Loading

0 comments on commit e80cde0

Please sign in to comment.