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

Update diagnostic message according to standard tool parsing. #151

Merged
merged 1 commit into from May 17, 2019

Conversation

warrenseine
Copy link
Contributor

MSBuild and many other tools / IDEs will correctly parse the message if ":" is added right after the error code.

See https://blogs.msdn.microsoft.com/msbuild/2006/11/02/msbuild-visual-studio-aware-error-messages-and-message-formats/ for reference.

MSBuild and many other tools / IDEs will correctly parse the message if ":" is added right after the error code.

See https://blogs.msdn.microsoft.com/msbuild/2006/11/02/msbuild-visual-studio-aware-error-messages-and-message-formats/ for reference.
@warrenseine
Copy link
Contributor Author

For reference, I'm currently sanitizing the error output with stripAnsi(e.message).replace(/(TS[0-9]+)/, "$1:")) where e is the Error rollup raises containing the printed diagnostic of the TypeScript compiler error.

@ezolenko ezolenko merged commit 5b629f1 into ezolenko:master May 17, 2019
@ezolenko
Copy link
Owner

Thanks, I'll make a release soonish

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

Successfully merging this pull request may close these issues.

None yet

2 participants