Skip to content

ci: reuse existing winget fork for publication#86

Merged
muscariello merged 3 commits intomainfrom
fix/winget-existing-fork-sync
Apr 21, 2026
Merged

ci: reuse existing winget fork for publication#86
muscariello merged 3 commits intomainfrom
fix/winget-existing-fork-sync

Conversation

@muscariello
Copy link
Copy Markdown
Member

Summary

Update the WinGet publication workflow to reuse an existing agntcy/winget-pkgs fork instead of calling GitHub endpoints the project app cannot use.

What Changed

  • require a pre-created agntcy/winget-pkgs fork instead of attempting to create one on demand
  • replace gh repo sync with a clone of the existing fork plus a fetch from upstream/master
  • make upstream remote setup idempotent for fork clones
  • stop hard-coding the fork default branch when cloning
  • document the pre-created fork requirement in CONTRIBUTING.md

Validation

  • reproduced the pre-fix failure on main: run 24722363681 failed on merge-upstream
  • validated the updated workflow on the branch: run 24722727851 completed successfully
  • successful branch run confirmed the existing-fork flow works end to end and exited cleanly because AGNTCY.shadictl 0.1.1 is already up to date on upstream/master

Refs #69

Signed-off-by: Luca Muscariello <muscariello@ieee.org>
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
Signed-off-by: Luca Muscariello <muscariello@ieee.org>
@muscariello muscariello requested a review from a team as a code owner April 21, 2026 12:40
@muscariello muscariello requested a review from micpapal April 21, 2026 12:40
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@muscariello muscariello merged commit 5227081 into main Apr 21, 2026
12 checks passed
@muscariello muscariello deleted the fix/winget-existing-fork-sync branch April 21, 2026 13:19
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