Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

fix: Fix non-top-level requires #37

Merged
merged 1 commit into from
Feb 11, 2020
Merged

fix: Fix non-top-level requires #37

merged 1 commit into from
Feb 11, 2020

Conversation

chrisbreiding
Copy link
Collaborator

Closes #36

@chrisbreiding
fix non-top-level requires
fa2a660 
revert @babel/plugin-transform-modules-commonjs to babel-plugin-add-module-exports
bahmutov
bahmutov approved these changes Feb 11, 2020
View reviewed changes
Copy link
Contributor

@bahmutov bahmutov left a comment

Choose a reason for hiding this comment

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

back to 3rd party module I guess

@chrisbreiding chrisbreiding changed the title Fix non-top-level requires fix: Fix non-top-level requires Feb 11, 2020
@chrisbreiding chrisbreiding merged commit 642a671 into master Feb 11, 2020
@chrisbreiding chrisbreiding deleted the issue-36-non-top-level-requires branch February 11, 2020 16:35
@chrisbreiding
Copy link
Collaborator Author

🎉 This PR is included in version 2.1.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Omrisnyk Omrisnyk mentioned this pull request Nov 28, 2023
Open
@Omrisnyk Omrisnyk mentioned this pull request Dec 7, 2023
Open
@Omrisnyk Omrisnyk mentioned this pull request Dec 19, 2023
Open
@Omrisnyk Omrisnyk mentioned this pull request May 14, 2024
Open
@Omrisnyk Omrisnyk mentioned this pull request May 22, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bahmutov bahmutov bahmutov approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Non-top-level requires not working
2 participants
@chrisbreiding @bahmutov