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

[heft] Add support for mapping TS Error codes to warnings in the typescript.json config file. #3540

Open
iclanton opened this issue Jul 18, 2022 · 2 comments
Assignees
Labels
enhancement The issue is asking for a new feature or design change
Projects

Comments

@iclanton
Copy link
Member

Summary

As mentioned in this issue (#3522), it would be useful to have this feature implemented: https://github.com/microsoft/rushstack/blob/main/apps/heft/src/plugins/TypeScriptPlugin/TypeScriptBuilder.ts#L780

Repro steps

N/A

Details

https://github.com/microsoft/rushstack/blob/main/apps/heft/src/plugins/TypeScriptPlugin/TypeScriptBuilder.ts#L780

Standard questions

Please answer these questions to help us investigate your issue more quickly:

Question Answer
@rushstack/heft version? Current
Operating system? Any
Would you consider contributing a PR? Yes
Node.js version (node -v)? Any
@iclanton iclanton added this to Low priority in Bug Triage Jul 18, 2022
@octogonz
Copy link
Collaborator

+1 I'm surprised the compiler itself doesn't provide this feature

@octogonz octogonz added the enhancement The issue is asking for a new feature or design change label Jul 20, 2022
@Faithfinder
Copy link
Contributor

I've read somewhere that TS error codes aren't considered "stable". Found this comment right now: microsoft/TypeScript#22011 (comment) but I think I saw a more elaborate one somewhere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue is asking for a new feature or design change
Projects
Status: Low priority
Bug Triage
  
Low priority
Development

No branches or pull requests

5 participants
@D4N14L @octogonz @iclanton @Faithfinder and others