Skip to content

Remove dbt-artifacts-parser from 7 day exclusion#777

Merged
jairus-m merged 2 commits into
mainfrom
jairus/remove-dbt-artifact-parser-exclusion
May 14, 2026
Merged

Remove dbt-artifacts-parser from 7 day exclusion#777
jairus-m merged 2 commits into
mainfrom
jairus/remove-dbt-artifact-parser-exclusion

Conversation

@jairus-m
Copy link
Copy Markdown
Collaborator

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

Summary

Added this exclusion as short-term config as i created a change in dbt-artifacts-parser repo to support some work in the dbt-mcp. A fresh release was coordinated with that project to help unblock me.

In retrospect, I should've just waited the 7 days to remove the exclusion config before merging the PR (did not register with me). Conveniently, by around 6p PT TODAY it will be 7 days since release so uv sync should works properly with this exclusion removed.

Screenshot 2026-05-13 at 3 14 22 PM

@jairus-m jairus-m requested review from a team, b-per and jasnonaz as code owners May 13, 2026 22:05
@jairus-m jairus-m merged commit 1fdb071 into main May 14, 2026
6 of 9 checks passed
@jairus-m jairus-m deleted the jairus/remove-dbt-artifact-parser-exclusion branch May 14, 2026 04:48
jairus-m added a commit that referenced this pull request May 14, 2026
…st (#778)

## Summary
Failure surfaced in #777 's integration test -
https://github.com/dbt-labs/dbt-mcp/actions/runs/25841708919/job/75928628173
- tho unrelated to PR.

4 failures total from that above integration test:
- 3 due to staging env error (unrelated to code and resolved after
retry)
- 1 due to need for unpacking tuple result after #774 (this PR fixes
this one only)
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.

2 participants