Skip to content

Fix exit status for failed publishes - #270

Closed
Companion wants to merge 1 commit into
UpliftGames:mainfrom
Companion:fix/publish-error-exit-status
Closed

Companion wants to merge 1 commit into
UpliftGames:mainfrom
Companion:fix/publish-error-exit-status

Conversation

@Companion

@Companion Companion commented Oct 4, 2026 •

Copy link
Copy Markdown

Fixes #259

Fix

Return an error when the registry rejects a publish so the CLI exits unsuccessfully and writes the status and response body to stderr

Successful publishes keep their existing message and exit status

Tests

  • Added real CLI regressions using isolated local registries for 401 422 and 500 responses
  • All three error regressions fail before the fix and pass after it
  • Verified 200 201 and 204 responses still succeed
  • Full all features workspace suite passed with 65 tests passed and one existing test ignored
  • Formatting build and Clippy with all features and all targets passed
  • Clippy reports existing warnings outside the changed files

@UpliftMacaw UpliftMacaw mentioned this pull request Oct 12, 2026
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.

Publish errors are not distinguishable (no exit code / stderr print)

1 participant