Skip to content

feat: beautify CLI update available notification and link to release notes #7237

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

serhalp
Copy link
Collaborator

@serhalp serhalp commented Apr 24, 2025

Summary

This polishes the look and brand of the CLI update available notification and adds a link to release notes.

When looking at the screenshots below, keep in mind they just show my own terminal background and colour configuration. These can vary, but we do use libraries that automatically adapt to available color space, support for OSC 8 terminal links, etc.

Before

before

After

Screenshot 2025-04-24 at 16 55 36

Copy link

📊 Benchmark results

Comparing with 759cc88

  • Dependency count: 1,150 (no change)
  • Package size: 280 MB ⬆️ 0.00% increase vs. 759cc88
  • Number of ts-expect-error directives: 426 (no change)

@serhalp serhalp marked this pull request as ready for review April 24, 2025 21:22
@serhalp serhalp requested a review from a team as a code owner April 24, 2025 21:22
const { createMainCommand } = await import('../dist/commands/main.js')
const { logError } = await import('../dist/utils/command-helpers.js')
const { default: getPackageJson } = await import('../dist/utils/get-cli-package-json.js')
const { runProgram } = await import('../dist/utils/run-program.js')

const pkg = await getPackageJson()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

unrelated, but I moved this into the try-catch since it's only for the update notification stuff that we want to degrade gracefully

@serhalp serhalp merged commit 0bf2db7 into main Apr 25, 2025
52 checks passed
@serhalp serhalp deleted the serhalp/cpla-2681-timebox-make-the-cli-update-call-to-action-more-enticing branch April 25, 2025 16:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants