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
feat(language-service): support to fix invalid banana in box #47393
Conversation
07cc6a0
to
c3e4bfa
Compare
c3e4bfa
to
9e7497c
Compare
Sorry, didn't see this. I'll review tomorrow morning. |
// cc @atscott |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌 Awesome. LGTM
Can you also add the "fixes #44941" note to the commit message?
packages/language-service/src/codefixes/fix_invalid_banana_in_box.ts
Outdated
Show resolved
Hide resolved
packages/language-service/src/codefixes/fix_invalid_banana_in_box.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed-for: language-service
Looks like Andrew beat me to it. Thanks for implementing this :) |
7c30df6
to
f47df65
Compare
The Angular compiler will report the invalid banana in box, this code fixes will try to fix the error and all the same errors in the selected file. Fixes angular#44941
f47df65
to
e89c609
Compare
This PR was merged into the repository by commit e7ee53c. |
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. |
The Angular compiler will report the invalid banana in box, this code fixes will try to fix the error and all the same errors in the selected file.
fixes #44941
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?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information