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.

Sign up for GitHub

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

Jump to bottom

Support for "require-module"? #52

Closed
moshest opened this issue Aug 18, 2017 · 5 comments
Closed

Support for "require-module"? #52

moshest opened this issue Aug 18, 2017 · 5 comments
Labels
enhancement 👑 New feature or request
Milestone
v2.11.0

Comments

@moshest
Copy link

moshest commented Aug 18, 2017 •
edited
Loading

The module property is like the main property but for ES6 Import-Export modules.
@tclindner
Copy link
Owner

Hey @moshest is this officially supported? I don't see it in npm's package.json documentation.

@moshest
Copy link
Author

moshest commented Aug 22, 2017 •
edited
Loading

Module is been used by Rollup & webpack bundlers.

You can see here: https://stackoverflow.com/a/42817320/518153
and here: https://medium.com/webpack/webpack-and-rollup-the-same-but-different-a41ad427058c

@tclindner
Copy link
Owner

Thank you for the information, @moshest! Marking this as an enhancement.

@tclindner tclindner added the enhancement 👑 New feature or request label Aug 24, 2017
@tclindner tclindner added this to the v2.11.0 milestone Sep 2, 2017
tclindner pushed a commit that referenced this issue Sep 2, 2017
Closes #52 - add new require-module rule
4dd83dc
@tclindner
Copy link
Owner

Hey @moshest - v2.11.0 has been released with this rule! We can keep an eye on the proposal and adjust the rule, if needed!

@moshest
Copy link
Author

moshest commented Sep 2, 2017

Great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement 👑 New feature or request
Projects
None yet
Milestone
v2.11.0
Development

No branches or pull requests
2 participants
@moshest @tclindner