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

Remove JSXNamespacedName from valid CallExpression args #16421

Merged

Conversation

nicolo-ribaudo
Copy link
Member

Q                       A
Fixed Issues? Fixes #16420
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

JSXNamespacedName is used for namespaced XML properties in things like <div namespace:name="value">, but fn(namespace:name) doesn't make any sense.

@nicolo-ribaudo nicolo-ribaudo added the PR: Internal 🏠 A type of pull request used for our changelog categories label Apr 8, 2024
@babel-bot
Copy link
Collaborator

babel-bot commented Apr 8, 2024

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/56710

@nicolo-ribaudo nicolo-ribaudo force-pushed the classexpr-remove-jsx-namespaced branch from c565c29 to 063c02f Compare April 8, 2024 12:28
@nicolo-ribaudo nicolo-ribaudo merged commit 33d8d7d into babel:main Apr 12, 2024
50 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the classexpr-remove-jsx-namespaced branch April 12, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: types PR: Internal 🏠 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CallExpression should not accept JSXNamespacedName as an argument
3 participants