FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Move `ensureFunctionName` to `NodePath.prototype` by nicolo-ribaudo · Pull Request #16658 · babel/babel · GitHub

/ babel Public

Move ensureFunctionName to NodePath.prototype - #16658

Merged
nicolo-ribaudo merged 3 commits into
babel:mainfrom
nicolo-ribaudo:remove-cycles-4
Jul 26, 2024
Merged

nicolo-ribaudo merged 3 commits into
babel:mainfrom
nicolo-ribaudo:remove-cycles-4

Conversation

nicolo-ribaudo commented Jul 17, 2024
edited
Loading

Copy link
Copy Markdown
Member
Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This should be the last non-peerDependencies cycle between packages :)

nicolo-ribaudo added PR: New Feature 🚀 A type of pull request used for our changelog categories pkg: traverse labels Jul 17, 2024

babel-bot commented Jul 17, 2024
edited
Loading

Copy link
Copy Markdown
Collaborator

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

nicolo-ribaudo requested a review from JLHwung July 25, 2024 18:50
nicolo-ribaudo force-pushed the feat-7.25.0/remove-cycles branch 2 times, most recently from bcac3c8 to 62b12b0 Compare July 26, 2024 10:30
nicolo-ribaudo deleted the branch babel:main July 26, 2024 12:21
nicolo-ribaudo changed the base branch from feat-7.25.0/remove-cycles to main July 26, 2024 12:22
nicolo-ribaudo merged commit d364545 into babel:main Jul 26, 2024
nicolo-ribaudo deleted the remove-cycles-4 branch July 26, 2024 15:58

Copy link
Copy Markdown

Hello, because of this change we're seeing a breaking change: value.ensureFunctionName is not a function.

Can you please let us know how we fix this?

github-actions Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Nov 2, 2024
github-actions Bot locked as resolved and limited conversation to collaborators Nov 2, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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: traverse PR: New Feature 🚀 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL