Upgrade pre-commit hooks (#176)

Author: yu-iskw

.pre-commit-config.yaml

@@ -24,7 +24,7 @@ repos:
       - id: check-yaml
       - id: detect-private-key
   - repo: https://github.com/rhysd/actionlint
-    rev: v1.7.7
+    rev: v1.7.8
     hooks:
       - id: actionlint
 #  - repo: https://github.com/asottile/reorder_python_imports
@@ -36,14 +36,15 @@ repos:
     hooks:
       - id: setup-cfg-fmt
   - repo: https://github.com/pycqa/isort
-    rev: 6.1.0
+    rev: 7.0.0
     hooks:
       - id: isort
         name: isort (python)
   - repo: https://github.com/pycqa/pylint
-    rev: v3.3.9
+    rev: v4.0.0
     hooks:
       - id: pylint
+        exclude: (dbt_artifacts_parser/parsers/)
 #  - repo: https://github.com/pre-commit/mirrors-mypy
 #    rev: 'v1.4.1'
 #    hooks:
@@ -53,7 +54,12 @@ repos:
     rev: v0.11.0.1
     hooks:
       - id: shellcheck
-  - repo: https://github.com/rhysd/actionlint
-    rev: v1.7.7
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    # Ruff version.
+    rev: v0.14.0
     hooks:
-      - id: actionlint-docker
+      # Run the linter.
+      - id: ruff-check
+        args: [ --fix ]
+      # Run the formatter.
+      - id: ruff-format

.style.yapf

@@ -14,7 +14,7 @@ setup-pre-commit:
 
 # Check all the coding style.
 .PHONY: lint
-lint: run-pre-commit lint-shell lint-python
+lint: run-pre-commit
 
 # Run the pre-commit hooks.
 .PHONY: run-pre-commit
@@ -26,25 +26,6 @@ run-pre-commit:
 update-pre-commit:
 	pre-commit autoupdate
 
-# Check the coding style for the shell scripts.
-.PHONY: lint-shell
-lint-shell:
-	shellcheck ./dev/*.sh
-
-# Check the coding style for the python files.
-.PHONY: lint-python
-lint-python:
-	bash ./dev/lint_python.sh
-
-# Format source codes
-.PHONY: format
-format: format-python
-
-# Format python codes
-.PHONY: format-python
-format-python:
-	bash ./dev/format_python.sh
-
 # Run the unit tests.
 .PHONY: test
 test:

Makefile

@@ -74,18 +74,18 @@ def parse_catalog_v1(catalog: dict) -> CatalogV1:
 def parse_manifest(
     manifest: dict,
 ) -> Union[
-        ManifestV1,
-        ManifestV2,
-        ManifestV3,
-        ManifestV4,
-        ManifestV5,
-        ManifestV6,
-        ManifestV7,
-        ManifestV8,
-        ManifestV9,
-        ManifestV10,
-        ManifestV11,
-        ManifestV12,
+    ManifestV1,
+    ManifestV2,
+    ManifestV3,
+    ManifestV4,
+    ManifestV5,
+    ManifestV6,
+    ManifestV7,
+    ManifestV8,
+    ManifestV9,
+    ManifestV10,
+    ManifestV11,
+    ManifestV12,
 ]:
     """Parse manifest.json
 
@@ -198,6 +198,7 @@ def parse_manifest_v9(manifest: dict) -> ManifestV9:
         return ManifestV9(**manifest)
     raise ValueError("Not a manifest.json v9")
 
+
 def parse_manifest_v10(manifest: dict) -> ManifestV10:
     """Parse manifest.json ver.10"""
     dbt_schema_version = get_dbt_schema_version(artifact_json=manifest)
@@ -227,8 +228,9 @@ def parse_manifest_v12(manifest: dict) -> ManifestV12:
 #
 def parse_run_results(
     run_results: dict,
-) -> Union[RunResultsV1, RunResultsV2, RunResultsV3, RunResultsV4,
-           RunResultsV5, RunResultsV6]:
+) -> Union[
+    RunResultsV1, RunResultsV2, RunResultsV3, RunResultsV4, RunResultsV5, RunResultsV6
+]:
     """Parse run-results.json
 
     Args:
@@ -284,6 +286,7 @@ def parse_run_results_v4(run_results: dict) -> RunResultsV4:
         return RunResultsV4(**run_results)
     raise ValueError("Not a run-results.json v4")
 
+
 def parse_run_results_v5(run_results: dict) -> RunResultsV5:
     """Parse run-results.json v5"""
     dbt_schema_version = get_dbt_schema_version(artifact_json=run_results)
@@ -299,6 +302,7 @@ def parse_run_results_v6(run_results: dict) -> RunResultsV6:
         return RunResultsV6(**run_results)
     raise ValueError("Not a run-results.json v6")
 
+
 #
 # sources
 #

dbt_artifacts_parser/parser.py

@@ -12,21 +12,21 @@
 
 class Metadata(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
     dbt_schema_version: Optional[str] = None
-    dbt_version: Optional[str] = '1.9.0b2'
+    dbt_version: Optional[str] = "1.9.0b2"
     generated_at: Optional[str] = None
     invocation_id: Optional[str] = None
     env: Optional[Dict[str, str]] = None
 
 
 class Metadata1(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
     type: str
-    schema_: str = Field(..., alias='schema')
+    schema_: str = Field(..., alias="schema")
     name: str
     database: Optional[str] = None
     comment: Optional[str] = None
@@ -35,7 +35,7 @@ class Metadata1(BaseParserModel):
 
 class Columns(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
     type: str
     index: int
@@ -45,7 +45,7 @@ class Columns(BaseParserModel):
 
 class Stats(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
     id: str
     label: str
@@ -56,30 +56,30 @@ class Stats(BaseParserModel):
 
 class Nodes(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
-    metadata: Metadata1 = Field(..., title='TableMetadata')
+    metadata: Metadata1 = Field(..., title="TableMetadata")
     columns: Dict[str, Columns]
     stats: Dict[str, Stats]
     unique_id: Optional[str] = None
 
 
 class Sources(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
-    metadata: Metadata1 = Field(..., title='TableMetadata')
+    metadata: Metadata1 = Field(..., title="TableMetadata")
     columns: Dict[str, Columns]
     stats: Dict[str, Stats]
     unique_id: Optional[str] = None
 
 
 class CatalogV1(BaseParserModel):
     model_config = ConfigDict(
-        extra='forbid',
+        extra="forbid",
     )
-    metadata: Metadata = Field(..., title='CatalogMetadata')
+    metadata: Metadata = Field(..., title="CatalogMetadata")
     nodes: Dict[str, Nodes]
     sources: Dict[str, Sources]
     errors: Optional[List[str]] = None
-    field_compile_results: Any = Field(None, alias='_compile_results')
+    field_compile_results: Any = Field(None, alias="_compile_results")