Skip to content

Commit

Permalink
chore: Remove noqa (fixed in sqlfluff)
Browse files Browse the repository at this point in the history
  • Loading branch information
jpmckinney committed Oct 17, 2024
1 parent 5428618 commit d63f24c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ repos:
args: [requirements_dev.in, -o, requirements_dev.txt]
files: ^requirements(_dev)?\.(in|txt)$
- repo: https://github.com/sqlfluff/sqlfluff
rev: 3.2.2
rev: 3.2.4
hooks:
- id: sqlfluff-lint
- id: sqlfluff-fix
2 changes: 1 addition & 1 deletion sql/middle/agg_parties.sql
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ WITH parties_role_counts AS (
count(*) AS total_parties_roles
FROM
parties_summary
-- https://github.com/sqlfluff/sqlfluff/issues/4623#issuecomment-2401209085
-- https://github.com/sqlfluff/sqlfluff/issues/4623#issuecomment-2401209085 >3.2.4
CROSS JOIN jsonb_array_elements_text(roles) AS role -- noqa: AL05
GROUP BY
id,
Expand Down
5 changes: 2 additions & 3 deletions sql/middle/parties_summary.sql
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,10 @@ SELECT
ELSE
'[]'::jsonb
END
-- https://github.com/sqlfluff/sqlfluff/issues/4623#issuecomment-2401209085
-- https://github.com/sqlfluff/sqlfluff/issues/4623#issuecomment-2401209085 >3.2.4
) AS additional_identifier -- noqa: AL05
WHERE
-- https://github.com/sqlfluff/sqlfluff/pull/6323
additional_identifier ?& ARRAY['scheme', 'id'] -- noqa: LT02,PRS
additional_identifier ?& ARRAY['scheme', 'id']
) AS additionalidentifiers_ids,
CASE
WHEN jsonb_typeof(value -> 'additionalIdentifiers') = 'array'
Expand Down

0 comments on commit d63f24c

Please sign in to comment.