chore: update dbt-data-reliability to include elementary_extra_indexes support#2265
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
👋 @haritamar |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughReplace the published ChangesPackage Source Migration
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@elementary/monitor/dbt_project/packages.yml`:
- Line 5: The dbt dependency revision for dbt-data-reliability is out of sync:
packages.yml pins revision 3dc104a13cda8dd4a0fbcec41a97706acb110643 while
package-lock.yml still references d4a280c3f97c33c156875302d6d294c0792c234e;
regenerate the lock so both files match by running dbt deps --lock against the
elementary/monitor/dbt_project (this will update package-lock.yml to the
revision in packages.yml).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: cc4b35c1-c0c1-465d-8ba7-87f52d7f9026
📒 Files selected for processing (1)
elementary/monitor/dbt_project/packages.yml
…s support Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
fb3c5a7 to
dff303a
Compare
Summary
Updates
packages.ymlto point to the merged commit ondbt-data-reliabilitymaster (3dc104a1) which includes theelementary_extra_indexesconfig var support (PR #1019).This is a dependency-chain update — the elementary-internal cloud runner passes
elementary_extra_indexesvia dbt vars to inject custom indexes (e.g.,(created_at, resource_type)ondbt_run_results) into synced schema tables. This PR picks up that macro support so elementary-internal can reference it.Merge order:
dbt-data-reliability #1019(merged) → this PR → elementary-internal #6500Link to Devin session: https://app.devin.ai/sessions/278ceddbe6d24dbea36a83ecc232f3b6
Requested by: @haritamar
Summary by CodeRabbit