Skip to content

feat(fonts): show subset in dev url#14459

Merged
florian-lefebvre merged 1 commit intomainfrom
feat/fonts-dev-url-subset
Oct 2, 2025
Merged

feat(fonts): show subset in dev url#14459
florian-lefebvre merged 1 commit intomainfrom
feat/fonts-dev-url-subset

Conversation

@florian-lefebvre
Copy link
Copy Markdown
Member

Changes

  • Unifont 0.6 exposed the subset on meta.subset
  • This PR updates the URLs generated in dev to include the subset when present, to ease debugging
  • Includes a refactor to mark properties as required

Testing

Unit tests updated

Docs

Changeset

@florian-lefebvre florian-lefebvre self-assigned this Sep 30, 2025
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 30, 2025

🦋 Changeset detected

Latest commit: c7294e3

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Sep 30, 2025
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Sep 30, 2025

CodSpeed Performance Report

Merging #14459 will not alter performance

Comparing feat/fonts-dev-url-subset (c7294e3) with main (667bd64)

Summary

✅ 6 untouched

Copy link
Copy Markdown
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

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

LGTM!

@florian-lefebvre florian-lefebvre merged commit 916f9c2 into main Oct 2, 2025
23 checks passed
@florian-lefebvre florian-lefebvre deleted the feat/fonts-dev-url-subset branch October 2, 2025 11:52
@astrobot-houston astrobot-houston mentioned this pull request Oct 2, 2025
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