Skip to content

Conversation

florian-lefebvre
Copy link
Member

Changes

  • Updates the help command (fallback command) to use existing infra
  • Most of the PR is just moving stuff around since some code isn't specific to create-key anymore

Testing

  • Splits test in 2 to reflect the folder structure changes
  • No unit test is added for help as we already test the HelpDisplay. It would need an e2e test instead, which will come later

Docs

N/A, refactor

@florian-lefebvre florian-lefebvre self-assigned this Oct 21, 2025
Copy link

changeset-bot bot commented Oct 21, 2025

⚠️ No Changeset found

Latest commit: e35a84c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Oct 21, 2025
@florian-lefebvre florian-lefebvre marked this pull request as ready for review October 21, 2025 12:24
Copy link

codspeed-hq bot commented Oct 21, 2025

CodSpeed Performance Report

Merging #14595 will not alter performance

Comparing refactor/cli-help (e35a84c) with main (d5bdbc0)

Summary

✅ 6 untouched

@florian-lefebvre florian-lefebvre merged commit 63b2568 into main Oct 21, 2025
21 checks passed
@florian-lefebvre florian-lefebvre deleted the refactor/cli-help branch October 21, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants