Skip to content

Update DBT package version #2012

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

Merged
elazarlachkar merged 3 commits into from
Sep 17, 2025
Merged

Update DBT package version #2012

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9fe09fe
Update DBT package version
elazarlachkar Sep 17, 2025
56d8f23
Updated lock file as well
elazarlachkar Sep 17, 2025
13e81d5
Merge branch 'master' into ele-5074-index-test_result_rows
elazarlachkar Sep 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions elementary/monitor/dbt_project/package-lock.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@ packages:
- package: dbt-labs/dbt_utils
version: 0.8.6
- git: https://github.com/elementary-data/dbt-data-reliability.git
revision: cbabcb2c348be2b4c45ad72c3b94e7800a52a542
sha1_hash: 90d912df8d9c6a435f1936c7d93e4bf6c0efc4dd
revision: 0f59f9f5407db890780f225f6c1e362f38221133
sha1_hash: 0dde8f6be0323bd473ea26bae571cef6d53d8fba
Copy link

@coderabbitai coderabbitai bot Sep 17, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue

Fix YAML indentation for sha1_hash (breaks parsing).

sha1_hash must be nested under the same list item as git and revision. It’s currently top‑level due to missing indentation and will cause YAML parse errors / dbt to ignore the lock entry.

Apply this diff:

-sha1_hash: 0dde8f6be0323bd473ea26bae571cef6d53d8fba
+    sha1_hash: 0dde8f6be0323bd473ea26bae571cef6d53d8fba
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
sha1_hash: 0dde8f6be0323bd473ea26bae571cef6d53d8fba
sha1_hash: 0dde8f6be0323bd473ea26bae571cef6d53d8fba
🤖 Prompt for AI Agents 
In elementary/monitor/dbt_project/package-lock.yml around line 6, the sha1_hash
key is incorrectly placed at top level instead of nested under the same list
item as git and revision; indent sha1_hash so it is a child of the same mapping
that contains git and revision (match the indentation level of git/revision for
that list entry) so the YAML parser and dbt will associate the hash with the
correct package entry.

2 changes: 1 addition & 1 deletion elementary/monitor/dbt_project/packages.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ packages:
- package: dbt-labs/dbt_utils
version: [">=0.8.0", "<0.9.0"]
- git: https://github.com/elementary-data/dbt-data-reliability.git
revision: cbabcb2c348be2b4c45ad72c3b94e7800a52a542
revision: 0f59f9f5407db890780f225f6c1e362f38221133

# NOTE - for unreleased CLI versions we often need to update the package version to a commit hash (please leave this
# commented, so it will be easy to access)
Expand Down