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

Add @babel/traverse tests for re-exported bindings #12429

Merged
merged 1 commit into from Dec 6, 2020

Conversation

nicolo-ribaudo
Copy link
Member

@nicolo-ribaudo nicolo-ribaudo commented Dec 1, 2020

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

These two issues where actually fixed by #12395, but since they are on a completely different level I added two tests to make sure that they were actually fixed (since it's not obvious by looking at #12395)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 1, 2020

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 15cc8b7:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration

@babel-bot
Copy link
Collaborator

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

@nicolo-ribaudo
Copy link
Member Author

I'm merging this with a single review since it's just adding tests for an already fixed bug.

@nicolo-ribaudo nicolo-ribaudo merged commit b5b95f8 into babel:main Dec 6, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the test-reexports branch December 6, 2020 18:33
@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 Mar 8, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 8, 2021
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 pkg: traverse (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect global scope scope.rename with renamed re-export
3 participants