Skip to content

feat(ENG-15336): Add changelog option to AI Skills package push - #564

Open
pmcguckin-cloudsmith wants to merge 2 commits into
masterfrom
pmcguckin-cloudsmith/ENG-15336/add-changelog-to-skills-push
Open

pmcguckin-cloudsmith wants to merge 2 commits into
masterfrom
pmcguckin-cloudsmith/ENG-15336/add-changelog-to-skills-push

Conversation

@pmcguckin-cloudsmith

@pmcguckin-cloudsmith pmcguckin-cloudsmith commented Oct 27, 2026 •

Copy link
Copy Markdown

Description

Type of Change

  • New feature

Additional Notes

image image

@pmcguckin-cloudsmith
pmcguckin-cloudsmith force-pushed the pmcguckin-cloudsmith/ENG-15336/add-changelog-to-skills-push branch from da906ed to 233e991 Compare October 30, 2026 10:45
@pmcguckin-cloudsmith
pmcguckin-cloudsmith marked this pull request as ready for review October 30, 2026 10:46
Copilot AI balanced review requested due to automatic review settings October 30, 2026 10:46
@pmcguckin-cloudsmith
pmcguckin-cloudsmith requested a review from a team as a code owner October 30, 2026 10:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues were identified for the Skills-specific --changelog addition.

No open issues 🎉

What changed in this PR

Adds an optional --changelog argument to cloudsmith push skills, extending the existing package-push command for AI Skills releases.

Changes:

  • Registers --changelog exclusively for the Skills package format.
  • Adds tests for forwarding the changelog to validation and package creation, displaying it in Skills help, and excluding it from Raw help.
File Description
cloudsmith_cli/​cli/​tests/​test_push_skills_cli.py Adds command-level tests for changelog forwarding and option visibility.
cloudsmith_cli/​cli/​commands/​push.py Registers the Skills-specific --changelog option.

Review effort: Balanced

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants