Skip to content

Update pre-commit hooks & lint - #1077

Closed
wrridgeway wants to merge 4 commits into
masterfrom
update-pre-commit-hooks
Closed

Update pre-commit hooks & lint#1077
wrridgeway wants to merge 4 commits into
masterfrom
update-pre-commit-hooks

Conversation

@wrridgeway

@wrridgeway wrridgeway commented Jul 29, 2026

Copy link
Copy Markdown
Member

This PR updates pre-commit hooks for lintr and sqlfluff. I made as few changes as I could to actual code and noted all of them. Everything else is purely formatting.

@wrridgeway wrridgeway changed the title Update pre commit hooks Update pre commit hooks & lint Jul 29, 2026
FROM {{ source('iasworld', 'pardat') }} AS pardat
LEFT JOIN {{ source('tax', 'pin') }} AS pin
ON pardat.parid = pin.pin
LEFT JOIN {{ source('tax', 'pin') }} AS tax_pin

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed this alias.

),

wide AS (
long_to_wide AS (

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed this alias.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed aliases inside subqueries for this view.

Comment thread pyproject.toml
indented_then = true
indented_then_contents = false
allow_implicit_indents = true
implicit_indents = "allow"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our old syntax has been deprecated.

@wrridgeway wrridgeway changed the title Update pre commit hooks & lint Update pre-commit hooks & lint Jul 29, 2026
@wrridgeway wrridgeway closed this Jul 29, 2026
@wrridgeway
wrridgeway deleted the update-pre-commit-hooks branch July 29, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant