Skip to content

Remove JSXNamespacedName from valid CallExpression args - #16421

Merged
nicolo-ribaudo merged 1 commit into
babel:mainfrom
nicolo-ribaudo:classexpr-remove-jsx-namespaced
Apr 12, 2024
Merged

nicolo-ribaudo merged 1 commit into
babel:mainfrom
nicolo-ribaudo:classexpr-remove-jsx-namespaced

Conversation

@nicolo-ribaudo

Copy link
Copy Markdown
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

babel-bot commented Apr 8, 2024

Copy link
Copy Markdown
Collaborator

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
@nicolo-ribaudo
nicolo-ribaudo deleted the classexpr-remove-jsx-namespaced branch April 12, 2024 10:57
@github-actions github-actions Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Jul 13, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue 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