fix(sourcemaps): Display injection errors#2775
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
We have been generating these error messages, but we never print them, even though it seems like we probably mean to be printing them. Fixes #2774 Fixes [CLI-170](https://linear.app/getsentry/issue/CLI-170/sourcemap-injection-errors-never-displayed)
eead738 to
09e715a
Compare
There was a problem hiding this comment.
Nice, seems useful to print the errors.
I wonder, does the user know what do to in order to resolve the issue? Would it make sense to display some more info about the issue and/or how to resolve it?
Or is it something that CLI will be able to fix on its own?
Or another possibility, is this just warnings that we want to see for bug reports?
|
@lcian I guess we will see whether users are able to figure out what to do based on the error message. To be honest, I am not completely sure whether they will be able to figure it out. But, we can see and adjust the messages if they are too confusing for users. In any case, it definitely seems like a bug that these messages were not being displayed. |
We have been generating these error messages, but we never print them, even though it seems like we probably mean to be printing them.
Fixes #2774
Fixes CLI-170