Skip to content

Commit

Permalink
Do not print the message about unused placeholders
Browse files Browse the repository at this point in the history
  • Loading branch information
sultaniman committed Apr 3, 2024
1 parent b08b7d5 commit 17829b4
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions dlt/destinations/impl/filesystem/layout.py
Original file line number Diff line number Diff line change
Expand Up @@ -166,19 +166,9 @@ def check_layout(self) -> None:
# Build out the list of placeholder names
# which we will use to validate placeholders
# in a given config.layout template
not_used_params = []
if self.params:
for placeholder, _ in self.params.items():
self.allowed_placeholders.add(placeholder)
if placeholder not in self.layout_placeholders:
not_used_params.append(placeholder)
if not_used_params:
placeholders = ", ".join(not_used_params)
fmt.secho(
f"Placeholders specified in extra_params '{placeholders}' "
"are not used in layout template",
fg="blue",
)

# now collect all unknown placeholders from config.layout template
invalid_placeholders = [
Expand Down

0 comments on commit 17829b4

Please sign in to comment.