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

remove "main" field from package.json #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chrisbutler
Copy link

@chrisbutler chrisbutler commented Oct 29, 2017

hey @roramirez afaik, MMM doesn't actually do anything with the name or main fields in package.json (right @MichMich?)

and i think it's caused confusion for module authors who try to name their packages one thing and name their main .js file something else, and can't get things to load (i submitted PRs against two modules in the last few days where i had to fix this)

this would likely clear up some of the confusion

this doesn't actually do anything, and i think it's caused confusion for module authors who try to name their packages one thing and name their main .js file something else
@roramirez
Copy link
Owner

Sound reasonable your point, but why dont rewrite this thing by script to create_new module.

Can you explain me more about these module are you mention before. There some link where I can take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants