Skip to content

Comments

feat: update dbt-artifacts-parser dependency to 0.9.0#63

Merged
gaurpulkit merged 1 commit intomainfrom
AI-3422-datapilot-cli-dbt-project-health-can-not-parse-dbt-projects-with-exposures
Aug 1, 2025
Merged

feat: update dbt-artifacts-parser dependency to 0.9.0#63
gaurpulkit merged 1 commit intomainfrom
AI-3422-datapilot-cli-dbt-project-health-can-not-parse-dbt-projects-with-exposures

Conversation

@gaurpulkit
Copy link
Contributor

@gaurpulkit gaurpulkit commented Jul 31, 2025

Add support for ManifestV12 in DBTVersion union.
Screenshot 2025-07-31 at 7 09 29 PM


Important

Update dbt-artifacts-parser to 0.9.0 and add ManifestV12 support in manifest.py.

  • Dependencies:
    • Update dbt-artifacts-parser to version 0.9.0 in setup.py.
  • Versioning:
    • Bump version from 0.0.21 to 0.0.22 in setup.py.
  • Schema Support:
    • Add ManifestV12 to Manifest union in manifest.py.

This description was created by Ellipsis for 5637072. You can customize this summary. It will automatically update as commits are pushed.

…ncy to 0.9.0. Add support for ManifestV12 in DBTVersion union.
@gaurpulkit gaurpulkit requested a review from suryaiyer95 July 31, 2025 14:06
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 5637072 in 1 minute and 4 seconds. Click for details.
  • Reviewed 42 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 4 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. setup.py:16
  • Draft comment:
    Version bump to 0.0.22 is appropriate.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. setup.py:66
  • Draft comment:
    Dependency updated to dbt-artifacts-parser 0.9.0; ensure compatibility with this version.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. src/datapilot/core/platforms/dbt/schemas/manifest.py:21
  • Draft comment:
    New import for ManifestV12 added to support latest manifest; verify its API matches expectations.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
4. src/datapilot/core/platforms/dbt/schemas/manifest.py:32
  • Draft comment:
    ManifestV12 added to the Manifest union; ensure union ordering is as intended for type resolution.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_hKzzqRQTUq6gSgFw

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@gaurpulkit gaurpulkit merged commit 01e6a25 into main Aug 1, 2025
39 checks passed
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