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

Include npm build script? #418

Closed
lorensr opened this issue Aug 10, 2016 · 3 comments
Closed

Include npm build script? #418

lorensr opened this issue Aug 10, 2016 · 3 comments

Comments

@lorensr
Copy link
Contributor

lorensr commented Aug 10, 2016

I developed a component using this (worked great, thank you!), meant to be published on npm, but the first couple guides I've read on how to build the es6/jsx files were long and didn't wind up working for me. To simplify this process and encourage more people to publish components, perhaps you could add a build-for-npm script that:

  • compiled src/ to dist/
  • maybe prompts for package.json fields like description and main (eg dist/index.js)
  • logged "You can now run npm publish"
@gaearon
Copy link
Contributor

gaearon commented Aug 10, 2016

This package is meant for apps, not components, but I can see us providing this in the future.

@rogeliog
Copy link
Contributor

I think this relates to #423

@gaearon
Copy link
Contributor

gaearon commented Aug 11, 2016

Yea let's discuss there.

@gaearon gaearon closed this as completed Aug 11, 2016
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants