Skip to content

sqlfluff regex update #173

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

sqlfluff regex update #173

wants to merge 2 commits into from

Conversation

@benjaminwestern
Copy link
Collaborator

@benjaminwestern benjaminwestern commented Jul 19, 2025

Fix: SQLFluff Parsing for js {} Blocks

  • Issue: SQLFluff failed with a "templating/parsing error" when formatting .sqlx files containing js {} blocks.
  • Cause: The .sqlfluff configuration's regular expression (param_regex) did not recognize js as a valid block type.
  • Solution: The param_regex was updated to include js, and js = was added to the configuration.
  • Impact: Users can now correctly format files that contain JavaScript blocks. Testing confirms the fix with no regressions.

Not 100% certain this will fix #148 but maybe?
@benjaminwestern benjaminwestern marked this pull request as draft July 19, 2025 11:28
@benjaminwestern
Copy link
Collaborator Author

benjaminwestern commented Jul 19, 2025

@ashish10alex just spitballing on this one. Let me know what comes to mind when you see this :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@benjaminwestern