Skip to content

ci: gate action.yml Marketplace publishability#9

Merged
kexi merged 1 commit into
mainfrom
ci/marketplace-requirements-check
Jun 7, 2026
Merged

ci: gate action.yml Marketplace publishability#9
kexi merged 1 commit into
mainfrom
ci/marketplace-requirements-check

Conversation

@kexi

@kexi kexi commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • GitHub Marketplace publishing cannot be fully automated — the publish button has no API and requires accepting the Marketplace agreement in the web UI. What can be CI-guarded is that action.yml always stays in a publishable shape.
  • Add tests/marketplace_action_contract.rs (runs under the existing cargo test CI job) that pins the Marketplace metadata requirements: name, description, a branding block with icon + color, the color from GitHub's allowed set, and an icon shaped like a valid Feather name (and not on GitHub's blocklist). Same "pin both sides / fail at test time" spirit as the release and schema contract tests.
  • Document the one-time manual publish step — and the automatic per-release listing updates afterward — in the COMPATIBILITY.md release checklist.

Test plan

  • cargo test --test marketplace_action_contract passes (4 tests)
  • full cargo test + clippy -D warnings + fmt --check clean

Publishing the composite action to the GitHub Marketplace has fixed metadata
requirements (name, description, branding.icon/color from an allowed set) that
are easy to break in action.yml and only surface when someone tries to publish.
Add a contract test that pins those requirements so a regression fails at
`cargo test` time, alongside the existing release/schema contract tests.

The publish button itself has no API and needs the Marketplace agreement
accepted in the web UI, so document that one-time manual step (and the automatic
per-release updates afterward) in the release checklist.

Co-Authored-By: Claude <noreply@anthropic.com>
@kexi kexi merged commit bfe32c2 into main Jun 7, 2026
7 checks passed
@kexi kexi deleted the ci/marketplace-requirements-check branch June 7, 2026 06:45
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