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

Include URLs on API errors, not just CLI errors #2138

Open
guybedford opened this issue Apr 18, 2018 · 1 comment
Open

Include URLs on API errors, not just CLI errors #2138

guybedford opened this issue Apr 18, 2018 · 1 comment

Comments

@guybedford
Copy link
Contributor

The logic in https://github.com/rollup/rollup/blob/master/bin/src/run/batchWarnings.ts has a lot of useful error detail that would still be very useful to have in the API error scenarios, either in the message itself or as a property of the error.

Ideally we could move this "error metadata" into a place that is applied in all scenarios so that this isn't something CLI specific only.

@shellscape
Copy link
Contributor

@guybedford @lukastaegert I believe I saw that we're now including URLs for API errors. Is that true or does this still need to be addressed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants