You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This fails the safety check, because we first replace the -- and THEN get rid of whitespace. Need to flip that order.
-{#-- We must add a fake join key in case additional grouping variables are not provided --#}
-{#-- Redshift does not allow for dynamically created join conditions (e.g. full join on 1 = 1 --#}
-{#-- The same logic is used in equal_rowcount. In case of changes, maintain consistent logic --#}
-{% set group_by_columns = ['id_dbtutils_test_fewer_rows_than'] + group_by_columns %}
-{% set groupby_gb_cols = 'group by ' + group_by_columns|join(',') %}
+ {#- - We must add a fake join key in case additional grouping variables are not provided --#}
+ {#- - Redshift does not allow for dynamically created join conditions (e.g. full join on 1 = 1 --#}
+ {#- - The same logic is used in equal_rowcount. In case of changes, maintain consistent logic --#}
+ {% set group_by_columns = ["id_dbtutils_test_fewer_rows_than"] + group_by_columns %}
+ {% set groupby_gb_cols = "group by " + group_by_columns | join(",") %}
Additional context
What is the output of sqlfmt --version?
v0.18.3
The text was updated successfully, but these errors were encountered:
This fails the safety check, because we first replace the
--
and THEN get rid of whitespace. Need to flip that order.Additional context
What is the output of
sqlfmt --version
?v0.18.3
The text was updated successfully, but these errors were encountered: