Skip to content

chore(no-ticket): confidently say service is up to date - #483

Merged
coillteoir merged 1 commit into
masterfrom
jj-stack/chore-no-ticket-confidently-say-service-is-up-to-date-mmwovyvv
Oct 14, 2026
Merged

coillteoir merged 1 commit into
masterfrom
jj-stack/chore-no-ticket-confidently-say-service-is-up-to-date-mmwovyvv

Conversation

@coillteoir

Copy link
Copy Markdown
Contributor

the CLI has to believe in itself

@coillteoir
coillteoir requested a review from a team as a code owner October 14, 2026 10:17
Copilot AI balanced review requested due to automatic review settings October 14, 2026 10:17

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

The wording change is correctly implemented and covered by existing tests.

0 open findings

What changed in this PR

Clarifies the service-check output by removing uncertain wording.

Changes:

  • Reports the API library as definitively up to date.
  • Updates unit and integration test expectations.
File Description
cloudsmith_cli/​cli/​commands/​check.py Updates the status message.
cloudsmith_cli/​cli/​tests/​commands/​test_check.py Aligns assertions with the new message.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@coillteoir

Copy link
Copy Markdown
Contributor Author

Revision history

Version Changes from previous version Submitted commit
2 (current) 60b73f1e..3d93936c 3d93936c
1 Initial version 60b73f1e

Generated by jj-stack from this pull request's force-push history.

@coillteoir
coillteoir force-pushed the jj-stack/chore-no-ticket-confidently-say-service-is-up-to-date-mmwovyvv branch from 60b73f1 to 3d93936 Compare October 14, 2026 10:20
@coillteoir
coillteoir enabled auto-merge (squash) October 14, 2026 10:21
@coillteoir
coillteoir merged commit 009e333 into master Oct 14, 2026
18 checks passed
@coillteoir
coillteoir deleted the jj-stack/chore-no-ticket-confidently-say-service-is-up-to-date-mmwovyvv branch October 14, 2026 10:23
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.

3 participants