Add flake8 and isort to .pre-commit-config.yaml

This adds flake8 and isort to .pre-commit-config.yaml.  This way, they
will automatically be run on commit.

I chose the most recent available versions after verifying that they
don't cause any reports or changes in the current tree.

Internally at AdaCore, we also use a few flake8 plugins as well, so
perhaps that's another avenue for investigation.

v2: Also update the various file-selection clauses to pick up
gdb-gdb.py.in; include the isort change made to this file; and finally
add a comment about the exclusions from flake8.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
This commit is contained in:
Tom Tromey
2024-04-02 12:04:21 -06:00
parent b0ca740e8f
commit 16810e455f
2 changed files with 19 additions and 2 deletions

View File

@@ -5,4 +5,20 @@ repos:
rev: 24.3.0
hooks:
- id: black
files: 'gdb/.*'
types_or: [file]
files: 'gdb/.*\.py(\.in)?$'
- repo: https://github.com/pycqa/flake8
rev: 7.0.0
hooks:
- id: flake8
types_or: [file]
# Note this one is only run on gdb/python, not (for now) the
# test suite.
files: 'gdb/python/.*\.py(\.in)?$'
args: [--config, gdb/setup.cfg]
- repo: https://github.com/pycqa/isort
rev: 5.13.2
hooks:
- id: isort
types_or: [file]
files: 'gdb/.*\.py(\.in)?$'

View File

@@ -15,9 +15,10 @@
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
import gdb
import os.path
import gdb
class TypeFlag:
"""A class that allows us to store a flag name, its short name,