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
[ts] Enforce order for the override
modifier
#13209
[ts] Enforce order for the override
modifier
#13209
Conversation
nicolo-ribaudo
commented
Apr 26, 2021
Q | A |
---|---|
Fixed Issues? | Ref: microsoft/TypeScript#43660 |
Patch: Bug Fix? | |
Major: Breaking Change? | |
Minor: New Feature? | |
Tests Added + Pass? | Yes |
Documentation PR Link | |
Any Dependency Changes? | |
License | MIT |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/45621/ |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit a11b53d:
|
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.
Did this PR also introduce static override
support?
I need to update this PR for microsoft/TypeScript#43829
Yes 👍 |
a9f5e07
to
3c7aeb3
Compare
d7e8868
to
521dc30
Compare
CI caught a regression closely related to a TS bug I just found: microsoft/TypeScript#43836. I will only fix the regression partially, and wait for the TS bug to be fixed to understand what is the expected behavior. |
Oh it's not a regression, it's a bugfix lol |
The new tests in the allowlist are not related to this PR, they are just new TS tests. |
* [ts] Enforce order for the `override` modifier * generator * Add more checks * Update TS tests