Skip to content
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.

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

Add support for dbt 1.8.0 #5877

Closed
2 of 3 tasks
FrankTub opened this issue May 13, 2024 · 1 comment · Fixed by #5892
Closed
2 of 3 tasks

Add support for dbt 1.8.0 #5877

FrankTub opened this issue May 13, 2024 · 1 comment · Fixed by #5892
Labels
bug Something isn't working

Comments

@FrankTub
Copy link

Search before asking

  • I searched the issues and found no similar issues.

What Happened

Upgraded my virtualenv to sqlfluff 3.0.6 and dbt 1.8.0. Now running sqlfluff lint results in an error.

Expected Behaviour

SQLFluff to run.

Observed Behaviour

Output of sqlfluff lint:

TypeError: register_adapter() missing 1 required positional argument: 'mp_context'  

How to reproduce

Upgrade to mentioned package versions and run sqlfluff lint

Dialect

Postgres

Version

3.0.6

Configuration

[sqlfluff]
# Supported dialects https://docs.sqlfluff.com/en/stable/dialects.html
# Or run 'sqlfluff dialects'
dialect = postgres
# One of [raw|jinja|python|placeholder]
templater = dbt
# Comma separated list of rules to exclude, or None
exclude_rules = L031
# Below controls SQLFluff output, see max_line_length for SQL output
output_line_length = 160
max_line_length = 160
# Very large files can make the parser effectively hang.
# The more efficient check is the _byte_ limit check which
# is enabled by default. The previous _character_ limit check
# is still present for backward compatibility. This will be
# removed in a future version.
# Set either to 0 to disable.
large_file_skip_byte_limit = 50000

[sqlfluff:templater:dbt]
project_dir = .
profiles_dir = ~/.dbt

[sqlfluff:templater:jinja]
apply_dbt_builtins = True

[sqlfluff:rules]
single_table_references = unqualified

[sqlfluff:rules:capitalisation.keywords]
# Keywords
capitalisation_policy = lower

[sqlfluff:rules:capitalisation.identifiers]
# Unquoted identifiers
extended_capitalisation_policy = lower

[sqlfluff:rules:capitalisation.functions]
# Function names
extended_capitalisation_policy = lower

[sqlfluff:rules:capitalisation.literals]
# Null & Boolean Literals
capitalisation_policy = lower

[sqlfluff:rules:capitalisation.types]
# Data Types
extended_capitalisation_policy = lower

[sqlfluff:rules:layout.long_lines]
# Line length
ignore_comment_lines = True
ignore_comment_clauses = True

[sqlfluff:rules:references.keywords]
# Comma separated list of words to ignore for this rule
ignore_words = date,year_month,name,attribute

[sqlfluff:rules:structure.subquery]
# By default, allow subqueries in from clauses, but not join clauses
forbid_subquery_in = both

[sqlfluff:rules:ambiguous.join]
fully_qualify_join_types = both

[sqlfluff:rules:ambiguous.column_references]
# GROUP BY/ORDER BY column references
group_by_and_order_by_style = explicit

[sqlfluff:rules:references.special_chars]
# Special characters in identifiers
additional_allowed_characters = ""

[sqlfluff:rules:convention.casting_style]
# SQL type casting
preferred_type_casting_style = cast

[sqlfluff:layout:type:select_clause]
line_position = strict

[sqlfluff:layout:type:where_clause]
line_position = strict

[sqlfluff:layout:type:from_clause]
line_position = strict

[sqlfluff:layout:type:join_clause]
line_position = strict

[sqlfluff:layout:type:groupby_clause]
line_position = strict

[sqlfluff:layout:type:having_clause]
line_position = strict

Are you willing to work on and submit a PR to address the issue?

  • Yes I am willing to submit a PR!

Code of Conduct

@FrankTub FrankTub added the bug Something isn't working label May 13, 2024
@gwenwindflower
Copy link

gwenwindflower commented May 13, 2024

same error with same versions and particulars here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants