Skip to content

Conversation

@florian-lefebvre
Copy link
Member

@florian-lefebvre florian-lefebvre commented Nov 6, 2025

Changes

  • Extracted from refactor(cli): info #14609
  • As I worked on refactoring astro info, the scope unexpectedly exploded
  • This PR extracts some of the changes that are required by refactor(cli): info #14609 but can be done before:
    • Moves where the polyfill is applied (doesn't matter much, removed in v6)
    • Moves some abstractions as they are not local anymore (eg. not only for astro docs) but shared across several commands
    • Updates some abstractions in preparation for other usages

Testing

Updated, should pass

Docs

N/A, refactor

@florian-lefebvre florian-lefebvre self-assigned this Nov 6, 2025
@changeset-bot
Copy link

changeset-bot bot commented Nov 6, 2025

⚠️ No Changeset found

Latest commit: 0a783ad

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 Nov 6, 2025
@florian-lefebvre florian-lefebvre marked this pull request as ready for review November 6, 2025 13:01
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 6, 2025

CodSpeed Performance Report

Merging #14722 will not alter performance

Comparing refactor/cli-info-prep (0a783ad) with main (3d55c5d)

Summary

✅ 6 untouched

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