Skip to content

Add regression tests for regenerator#10223

Merged
nicolo-ribaudo merged 2 commits intobabel:masterfrom
nicolo-ribaudo:regenerator-tests
Mar 23, 2020
Merged

Add regression tests for regenerator#10223
nicolo-ribaudo merged 2 commits intobabel:masterfrom
nicolo-ribaudo:regenerator-tests

Conversation

@nicolo-ribaudo
Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo commented Jul 15, 2019

Q                       A
Fixed Issues? Closes #10193, closes #6923, closes #8525
Tests Added + Pass? Yes
Any Dependency Changes? regenerator updated
License MIT

while (1) switch (_context.prev = _context.next) {
case 0:
_actions.someAction = (bar, function () {
throw new Error('"' + "someAction" + '" is read-only.');
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong, waiting for facebook/regenerator#377

@babel-bot
Copy link
Copy Markdown
Collaborator

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

1 similar comment
@babel-bot
Copy link
Copy Markdown
Collaborator

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

@nicolo-ribaudo nicolo-ribaudo marked this pull request as ready for review July 19, 2019 17:12
"main": "lib/index.js",
"dependencies": {
"regenerator-transform": "^0.14.0"
"regenerator-transform": "^0.14.2"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like facebook/regenerator#377 is available in regenerator-transform:0.14.1 shouldn't this be 0.14.1?

Suggested change
"regenerator-transform": "^0.14.2"
"regenerator-transform": "^0.14.1"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right. Sorry. My bad.

@nicolo-ribaudo
Copy link
Copy Markdown
Member Author

@benjamn Could a new version of regenerator-transform including facebook/regenerator@ffc74cc be released? 🙏

@nicolo-ribaudo
Copy link
Copy Markdown
Member Author

Regenerator has been merged in #11191, I'll merge these tests.

@nicolo-ribaudo nicolo-ribaudo merged commit d23f0ae into babel:master Mar 23, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the regenerator-tests branch March 23, 2020 19:50
@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 Jun 23, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: tests outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Dependency ⬆️

Projects

None yet

3 participants