chore(release): enrich chocolatey nuspec metadata per moderation guidelines#510
Open
Matovidlo wants to merge 1 commit into
Open
chore(release): enrich chocolatey nuspec metadata per moderation guidelines#510Matovidlo wants to merge 1 commit into
Matovidlo wants to merge 1 commit into
Conversation
…elines Address the Guidelines flagged by chocolatey-ops on the keboola-cli2 0.66.1 moderation review (all Guidelines/Suggestions -- zero Requirements, so this is quality polish for the next published version, not a blocker): - add packageSourceUrl, projectSourceUrl, bugTrackerUrl - add releaseNotes (URL form -> no per-version edits, moderator-endorsed) - add copyright - stage a commented-out iconUrl (jsDelivr CDN) ready to enable once an icon PNG is committed; left off to avoid a 404 link Does NOT touch the workflow: the chocolatey push 403 on 0.71/0.72 is caused by 0.66.1 sitting in moderation with no approved version yet (documented chocolatey 403 trigger), which resolves when a moderator approves 0.66.1 -- not by anything in the release pipeline. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Author
|
@claude review |
There was a problem hiding this comment.
Pull request overview
Updates the Chocolatey nuspec metadata for keboola-cli2 to better align with Chocolatey moderation guidelines, improving the package’s listing quality without changing any runtime behavior.
Changes:
- Added repository/source metadata fields (
packageSourceUrl,projectSourceUrl,bugTrackerUrl). - Added URL-form
releaseNotesandcopyright. - Staged (commented-out)
iconUrlwith guidance for enabling it once an icon asset exists.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Matovidlo
marked this pull request as ready for review
July 21, 2026 09:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Enriches the Chocolatey package nuspec (
build/package/chocolatey/keboola-cli2.nuspec) to address the Guidelines flagged bychocolatey-opson thekeboola-cli20.66.1 moderation review.packageSourceUrl,projectSourceUrl,bugTrackerUrlreleaseNotesas a URL to the releases page (moderator-endorsed form → never needs per-version edits)copyrighticonUrl(jsDelivr CDN) ready to enable once an icon PNG is committed — left off deliberately to avoid a 404 link, which is worse than omitting itContext — why the Chocolatey channel is currently stuck (not this PR)
The 403 Forbidden on
choco pushfor 0.71.0 / 0.72.0 is not a pipeline or key bug. It's the documented Chocolatey behavior: a package with a version in moderation and no approved version rejects new-version pushes with 403.keboola-cli20.66.1 passed automated validation on 2026-07-15 with zero Requirements and is simply waiting in the human moderation queue. When a moderator approves 0.66.1, the block lifts and 0.71/0.72 can push (re-run the failed job, no new tag needed).The reviewer's comments were all Guidelines / Suggestions / Notes — explicitly "fix for next time," approvable without them. This PR is that quality polish; it ships with the next published version.
Impact analysis
build/package/chocolatey/keboola-cli2.nuspec: additive metadata only. XML validated well-formed.Test plan
python3 -c "import xml.dom.minidom; xml.dom.minidom.parse(...)"→ XML well-formed ✅iconUrlintentionally commented untilbuild/package/chocolatey/keboola.pngis committed; uncomment then.Change type
Chore — Chocolatey packaging metadata quality (no functional change)
Related
keboola-cli20.66.1 (chocolatey-ops, 2026-07-15)Deployment
Merge & picked up by the next release's chocolatey packaging step. No migration.
Rollback plan
Revert of this PR.
Post release support plan
None.