Skip to content

Commit

Permalink
Merge pull request #10 from finn-auto/DP-2431/check-target-schema-in-…
Browse files Browse the repository at this point in the history
…snapshots

[DP-2431] created new CI check for target_schema in snapshots
  • Loading branch information
ShahbalKhan authored Nov 28, 2023
2 parents ae224b8 + de61e8c commit cf9500e
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions pre_commit_dbt/check_snapshot_has_target_schema.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@


def validate_tags(
paths: Sequence[str], manifest: Dict[str, Any], tags: Sequence[str]
paths: Sequence[str], manifest: Dict[str, Any]
) -> int:
paths = get_missing_file_paths(paths, manifest)

Expand All @@ -40,13 +40,6 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
add_filenames_args(parser)
add_manifest_args(parser)

parser.add_argument(
"--tags",
nargs="+",
required=True,
help="A list of tags that models can have.",
)

args = parser.parse_args(argv)

try:
Expand All @@ -55,7 +48,7 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
print(f"Unable to load manifest file ({e})")
return 1

return validate_tags(paths=args.filenames, manifest=manifest, tags=args.tags)
return validate_tags(paths=args.filenames, manifest=manifest)


if __name__ == "__main__":
Expand Down

0 comments on commit cf9500e

Please sign in to comment.