Skip to content

Bump dbt-sl-sdk==0.13.3#785

Merged
jairus-m merged 3 commits into
mainfrom
jairus/bump-dbt-sl-sdk
May 22, 2026
Merged

Bump dbt-sl-sdk==0.13.3#785
jairus-m merged 3 commits into
mainfrom
jairus/bump-dbt-sl-sdk

Conversation

@jairus-m
Copy link
Copy Markdown
Collaborator

@jairus-m jairus-m commented May 22, 2026

Summary

Made a fix upstream in dbt-sl-sdk that fully addresses and closes #210 at the root. Bumping the version to incorporate these changes. More context here.

Also updated relevant prompt examples to remove old workaround.

Copilot AI review requested due to automatic review settings May 22, 2026 18:13
@jairus-m jairus-m requested review from a team, b-per and jasnonaz as code owners May 22, 2026 18:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository’s dependency lock to pick up an upstream fix in dbt-sl-sdk intended to resolve the entity GROUP BY behavior described in #210/#209.

Changes:

  • Update uv.lock to resolve dbt-sl-sdk to 0.13.3.
  • Add an “Under the Hood” changelog entry noting the dependency bump.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
uv.lock Updates the locked dbt-sl-sdk artifact to 0.13.3 (and updates lock metadata).
.changes/unreleased/Under the Hood-20260522-111026.yaml Adds an unreleased changelog entry for the dependency bump.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jairus-m jairus-m deployed to integration May 22, 2026 18:20 — with GitHub Actions Active
@jairus-m jairus-m enabled auto-merge (squash) May 22, 2026 18:23
@jairus-m jairus-m merged commit ca1e58d into main May 22, 2026
7 checks passed
@jairus-m jairus-m deleted the jairus/bump-dbt-sl-sdk branch May 22, 2026 18:24
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