Skip to content

Chocolatey: tag-pinned PNG icon, distinct source URL, manual republish job - #111

Merged
TheAbider merged 2 commits into
masterfrom
choco-icon-republish
Sep 9, 2026
Merged

Chocolatey: tag-pinned PNG icon, distinct source URL, manual republish job#111
TheAbider merged 2 commits into
masterfrom
choco-icon-republish

Conversation

@TheAbider

Copy link
Copy Markdown
Owner

Answers the moderator review on rackstack 1.123.1 (raw.githubusercontent.com icon not accepted) and the two guideline notes from the same review.

  • iconUrl -> https://cdn.jsdelivr.net/gh/TheAbider/RackStack@v{ver}/.github/assets/icon.png (tag-pinned CDN, PNG per the png/svg guideline). Stamped with the release version in both publish paths.
  • projectSourceUrl now differs from projectUrl; docsUrl points at the wiki.
  • RackStack.psd1 IconUri on the same CDN for consistency.
  • New workflow_dispatch job republish-chocolatey (input: version) rebuilds the .nupkg from the current templates and the published release's own hash, then re-pushes. The EXE is not rebuilt, so the hash, cosign signature, and antivirus submission for 1.123.1 stay valid. The normal test job is skipped on dispatch.
  • Pack verified locally against the stamped 1.123.1 nuspec. 5514/5514 structural tests.

…publish job

Moderator feedback on rackstack 1.123.1: raw.githubusercontent.com icons are
not accepted. The nuspec now points at the release's own tag on jsDelivr and
uses the PNG icon (the guideline prefers png/svg over ico). projectSourceUrl
now differs from projectUrl and docsUrl points at the wiki, clearing the two
guideline notes from the same review. RackStack.psd1 IconUri moved to the
same CDN for consistency.

ci.yml gains a workflow_dispatch job that rebuilds and re-pushes the
Chocolatey package for an EXISTING release from its published hash, so a
nuspec fix can land on a version already in moderation without rebuilding
the EXE or cutting a new version. The normal job is skipped on dispatch.
Pack verified locally against the stamped 1.123.1 nuspec; 5514/5514 tests.
@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@TheAbider
TheAbider merged commit 89be549 into master Sep 9, 2026
7 checks passed
@TheAbider
TheAbider deleted the choco-icon-republish branch September 9, 2026 04:57
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