Skip to content

Fix manifest download workflow selection#76

Closed
0xRobin wants to merge 1 commit into
mainfrom
robin/CUR2-1437-latest-manifest
Closed

Fix manifest download workflow selection#76
0xRobin wants to merge 1 commit into
mainfrom
robin/CUR2-1437-latest-manifest

Conversation

@0xRobin

@0xRobin 0xRobin commented Jun 25, 2026

Copy link
Copy Markdown

CUR2-1437
Changes dbt CI/deploy manifest downloads to look at the latest completed dbt_deploy run, not only the latest successful one. This lets failed deploys that still upload a current manifest advance state for later PR/deploy runs.

@cursor

cursor Bot commented Jun 25, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Workflow-only change to artifact selection; no dbt models or credentials logic touched.

Overview
Updates dbt CI and dbt deploy so dawidd6/action-download-artifact pulls prod-manifest-latest from the most recent completed dbt_deploy run on main, not only the latest successful one.

Both workflows now set workflow_conclusion: completed, with comments noting that failed deploys can still upload a current manifest (deploy already uploads with if: always()). That keeps PR state comparison and deploy’s own state:modified logic aligned with the newest compiled manifest even when a deploy job fails after compile/upload.

Reviewed by Cursor Bugbot for commit 7fdcee5. Configure here.

@0xRobin 0xRobin marked this pull request as ready for review June 25, 2026 14:36
@0xRobin 0xRobin closed this Jun 25, 2026
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.

1 participant