Skip to content

fix: align workflow hash owner for private registry#424

Open
j-nowak wants to merge 3 commits into
mainfrom
cursor/f39b17a4
Open

fix: align workflow hash owner for private registry#424
j-nowak wants to merge 3 commits into
mainfrom
cursor/f39b17a4

Conversation

@j-nowak
Copy link
Copy Markdown
Contributor

@j-nowak j-nowak commented May 12, 2026

Summary

  • load credentials for workflow hash to support private registries
  • use derived org owner for private/off-chain hash computation
  • add unit coverage for hash owner resolution

Test plan

  • go test ./cmd/workflow/hash
  • go test ./cmd/...

j-nowak and others added 3 commits May 12, 2026 14:16
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@j-nowak j-nowak changed the title test: cover workflow hash credential loading fix: align workflow hash owner for private registry May 12, 2026
@j-nowak j-nowak requested a review from timothyF95 May 12, 2026 14:38
@j-nowak j-nowak marked this pull request as ready for review May 12, 2026 14:39
@j-nowak j-nowak requested a review from a team as a code owner May 12, 2026 14:39
Copy link
Copy Markdown
Collaborator

@anirudhwarrier anirudhwarrier left a comment

Choose a reason for hiding this comment

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

workflow hash should work without login

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