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
Allow nullish extends in interfaceish #12920
Allow nullish extends in interfaceish #12920
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/43114/ |
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 2676750:
|
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.
Thanks!
Oh actually, could you add a test in https://github.com/babel/babel/blob/main/packages/babel-generator/test/index.js? |
Otherwise code such as ```ts t.interfaceDeclaration( t.identifier('id'), undefined, undefined, t.objectTypeAnnotation([]) ) ``` Will fail when priting with ` TypeError: unknown: Cannot read property 'length' of null` Despite nullish values being allowed in `t.interfaceDeclaration` definitions
Co-authored-by: Nicolò Ribaudo <nicolo.ribaudo@gmail.com>
d138249
to
2676750
Compare
Why yes, added. Hope it's in the right place |
I've allowed nullish
extends
in interfaceish, cause otherwise code such asWill fail when priting with
TypeError: unknown: Cannot read property 'length' of null
Despite nullish values being allowed in
t.interfaceDeclaration
definitions