Skip to content

Conversation

@mdesmet
Copy link
Collaborator

@mdesmet mdesmet commented Oct 2, 2025

This reverts commit f935b09.

@mdesmet mdesmet changed the title Revert "fix: use forked dbt-artifacts-parser (#69)" fix: use vendored dbt-artifacts-parser Oct 2, 2025
@ellipsis-dev
Copy link

ellipsis-dev bot commented Oct 2, 2025

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at help@ellipsis.dev


Generated with ❤️ by ellipsis.dev

@mdesmet mdesmet requested a review from suryaiyer95 October 2, 2025 23:39
Comment on lines +17 to +19
model_config = ConfigDict(
extra="forbid",
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we remove this?

Comment on lines +32 to +33
extra="forbid",
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you want to remove this?

@mdesmet mdesmet merged commit 91ccf35 into main Oct 2, 2025
39 checks passed
@mdesmet mdesmet deleted the fix/artifacts-parser-dbt-cloud branch October 2, 2025 23:55
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