Skip to content

Drop deprecated sqlfluff recurse config#21

Merged
tpoll merged 1 commit intometaplane:mainfrom
sambradbury:fix-sqlfluff-config
Jun 20, 2025
Merged

Drop deprecated sqlfluff recurse config#21
tpoll merged 1 commit intometaplane:mainfrom
sambradbury:fix-sqlfluff-config

Conversation

@sambradbury
Copy link
Contributor

Summary of Changes

Removes a config option deprecated by sqlfluff here.

Why Do We Need These Changes

Removing this option will prevent users from running into errors like the one below while trying to use sqlfluff in a dbt project where dbt-expectations is installed.

==== finding fixable violations ====

User Error: Config file PosixPath('/Users/myusername/Documents/dbt_project/dbt_packages/dbt_expectations/integration_tests/.sqlfluff') set an outdated config value core:recurse.

Removed as unused in production and unnecessary for debugging.

See https://docs.sqlfluff.com/en/stable/perma/configuration.html for more details.

Reviewers

@tpoll @gusvargas

@steveh-101
Copy link

Can this be reviewed and merged @tpoll @gusvargas? Hitting this error on latest sqlfluff

@tpoll tpoll merged commit 008e783 into metaplane:main Jun 20, 2025
2 checks passed
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.

3 participants