Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

dbt_utils.star: Remove trailing whitespace in the comment when no column is returned #960

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mcasara
Copy link

@mcasara mcasara commented Oct 22, 2024

resolves #959

Problem

The comment contains trailing whitespaces. The macro statements also generate trailing whitespaces, which fails sqlfluff's linting of the compiled file.

Solution

Remove the trailing whitespaces and use {%- ... -%} instead of {% ... %} in the macro statements.

Checklist

  • This code is associated with an issue which has been triaged and accepted for development.
  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the README.md (if applicable)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dbt_utils.star: Remove trailing whitespace in the comment when no column is returned
1 participant