Enable python linters and formatters (#5)

Author: abrisco

.bazelrc

@@ -37,7 +37,34 @@ build --incompatible_autoload_externally=
 build:verilator_lint --aspects=//verilator:verilator_lint_aspect.bzl%verilator_lint_aspect
 build:verilator_lint --output_groups=+verilator_lint_checks
 
-test --config=verilator_lint
+# Enable black for all targets in the workspace
+build:black --aspects=@rules_venv//python/black:defs.bzl%py_black_aspect
+build:black --output_groups=+py_black_checks
+
+# Enable isort for all targets in the workspace
+build:isort --aspects=@rules_venv//python/isort:defs.bzl%py_isort_aspect
+build:isort --output_groups=+py_isort_checks
+build --@rules_venv//python/isort:config=//:.isort.cfg
+
+# Enable mypy for all targets in the workspace
+build:mypy --aspects=@rules_venv//python/mypy:defs.bzl%py_mypy_aspect
+build:mypy --output_groups=+py_mypy_checks
+build --@rules_venv//python/mypy:config=//:.mypy.ini
+
+# Enable pylint for all targets in the workspace
+build:pylint --aspects=@rules_venv//python/pylint:defs.bzl%py_pylint_aspect
+build:pylint --output_groups=+py_pylint_checks
+build --@rules_venv//python/pylint:config=//:.pylintrc.toml
+
+# Define a config that enables all linter aspects.
+build:strict --config=verilator_lint
+build:strict --config=black
+build:strict --config=isort
+build:strict --config=pylint
+build:strict --config=mypy
+
+
+test --config=strict
 
 ###############################################################################
 ## Custom user flags

.isort.cfg

@@ -0,0 +1,2 @@
+[settings]
+profile = black

.mypy.ini

@@ -0,0 +1,11 @@
+# https://mypy.readthedocs.io/en/stable/config_file.html
+[mypy]
+
+# Improve strictness of checks
+strict = True
+
+# Improve logging
+pretty = True
+
+# Improve behavior in actions.
+explicit_package_bases = True

.pylintrc.toml

@@ -0,0 +1,15 @@
+[tool.pylint.main]
+
+# Allow loading of arbitrary C extensions. Extensions are imported into the
+# active Python interpreter and may run arbitrary code.
+unsafe-load-any-extension = true
+
+# Limit actions to using 1 core per action.
+jobs = 1
+
+disable = [
+    "fixme",                  # Developers should be allowed to leave TODO comments.
+    "wrong-import-position",  # isort is in charge of import ordering.
+    "wrong-import-order",     # isort is in charge of import ordering.
+    "line-too-long",          # Black is responsible for shortening where possible.
+]

BUILD.bazel

@@ -1,5 +1,8 @@
 exports_files([
+    ".isort.cfg",
+    ".mypy.ini",
+    ".pylintrc.toml",
+    "LICENSE",
     "MODULE.bazel",
     "version.bzl",
-    "LICENSE",
 ])

system_rdl/private/BUILD.bazel

@@ -6,12 +6,18 @@ current_system_rdl_peakrdl_toolchain(
     name = "current_system_rdl_peakrdl_toolchain",
 )
 
-py_binary(
+alias(
     name = "peakrdl",
+    actual = "peakrdl_wrapper",
+    visibility = ["//visibility:public"],
+)
+
+py_binary(
+    name = "peakrdl_wrapper",
     srcs = [
-        "peakrdl.py",
+        "peakrdl_wrapper.py",
     ],
-    main = "peakrdl.py",
+    main = "peakrdl_wrapper.py",
     deps = [
         ":current_system_rdl_peakrdl_toolchain",
     ],