Skip to content

Commit 4f37225

Browse files
benfdkingCopilot
andcommitted
Update sqlmesh/core/linter/rules/builtin.py
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 44ce4ad commit 4f37225

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

sqlmesh/core/linter/rules/builtin.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,8 @@ class NoUnregisteredExternalModels(Rule):
116116
def check_model(self, model: Model) -> t.Optional[RuleViolation]:
117117
depends_on = model.depends_on
118118

119-
# Ignore external models themselves, because either they are registered
120-
# if they are not, they will be caught as referenced in another model.
119+
# Ignore external models themselves, because either they are registered,
120+
# and if they are not, they will be caught as referenced in another model.
121121
if isinstance(model, ExternalModel):
122122
return None
123123

tests/core/linter/test_builtin.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
def test_no_unregistered_external_models(tmp_path, copy_to_temp_path) -> None:
77
"""
88
Tests that the linter correctly identifies unregistered external model dependencies.
9-
9+
1010
This test removes the `external_models.yaml` file from the sushi example project,
1111
enables the linter, and verifies that the linter raises a violation for a model
1212
that depends on unregistered external models.

0 commit comments

Comments
 (0)