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 error hints #3350

Merged
merged 2 commits into from
May 12, 2022
Merged

add error hints #3350

merged 2 commits into from
May 12, 2022

Conversation

FredKSchott
Copy link
Member

Changes

  • Solves a DX issue that the Solid + Builder teams were seeing, where it was unclear how to resolve a noExternal issue based on the console output.
  • Adds a larger "Hint" error metadata + formatter that you can use to report hints alongside errors.

Testing

  • TODO

Docs

  • N/A

@changeset-bot
Copy link

changeset-bot bot commented May 11, 2022

🦋 Changeset detected

Latest commit: 3e518f7

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

This PR includes changesets to release 1 package
Name Type
astro 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 May 11, 2022
@matthewp
Copy link
Contributor

code looks good, needs changeset.

@matthewp matthewp merged commit e48aa2f into main May 12, 2022
@matthewp matthewp deleted the add-error-hints branch May 12, 2022 16:30
@github-actions github-actions bot mentioned this pull request May 12, 2022
SiriousHunter pushed a commit to SiriousHunter/astro that referenced this pull request Feb 3, 2023
* add error hints

* chore: add changeset

Co-authored-by: Nate Moore <nate@skypack.dev>
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