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
build(compiler-cli): fix tsconfig.json #22722
Conversation
I removed the |
It's odd that my VS Code doesn't complain about this. It is obviously wrong. However if you delete the file, the editors will not understand that it's ok to use es2015 apis in the compiler-cli code. The proper fix would be to change the extends to "../tsconfig.json". Can you please update your PR to that? |
Thanks @IgorMinar for the feedback
I have updated my PR accordingly. Please let me know if this is not the way to go |
rebased on top of master and force pushed in order to have a new travis build (previous was failing due to timeout in |
thank you! |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Fixes #22721
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
In
compiler-cli/tsconfig.json
, there is a circularity issue (it is extending itself)Issue Number: N/A
#22721
What is the new behavior?
update
compiler-cli/tsconfig.json
introduced by c602563Does this PR introduce a breaking change?
Other information