Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add fontsource packages to always be noExternal #4073

Merged
merged 2 commits into from
Jul 27, 2022
Merged

Add fontsource packages to always be noExternal #4073

merged 2 commits into from
Jul 27, 2022

Conversation

matthewp
Copy link
Contributor

Changes

Testing

  • tested manually
  • we have a fontsource mocha test, but because of the monorepo it doesn't get loaded the same way as in a real app :/

Docs

N/A, bug fix

@changeset-bot
Copy link

changeset-bot bot commented Jul 27, 2022

🦋 Changeset detected

Latest commit: e6721c8

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

This PR includes changesets to release 12 packages
Name Type
astro Patch
@e2e/astro-component Patch
@e2e/error-react-spectrum Patch
@e2e/error-sass Patch
@e2e/errors Patch
@e2e/lit-component Patch
@e2e/preact-component Patch
@e2e/react-component Patch
@e2e/solid-component Patch
@e2e/svelte-component Patch
@e2e/e2e-tailwindcss Patch
@e2e/ts-resolution Patch

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 Jul 27, 2022
@matthewp matthewp merged commit 13b4f8a into main Jul 27, 2022
@matthewp matthewp deleted the fontsource branch July 27, 2022 19:48
@delucis
Copy link
Member

delucis commented Jul 27, 2022

Hey @matthewp what's the link between this PR and the issue it just closed? That issue seems to be about using a third party component not fontsource.

@astrobot-houston astrobot-houston mentioned this pull request Jul 27, 2022
@delucis
Copy link
Member

delucis commented Jul 27, 2022

Maybe should have closed #4064 instead?

@matthewp
Copy link
Contributor Author

Ah yeah, marked the wrong one, thanks for fixing.

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.

🐛 BUG: astro-embed integration fails due to .astro file extension error
3 participants