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

added pre-build 'meteor npm install' step #11

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

Conversation

andrew-wharton
Copy link

@andrew-wharton andrew-wharton commented Aug 25, 2016

#7 This is needed so that npm dependencies are available when building the application

This is needed so that npm dependencies are available when building the application
@andrew-wharton andrew-wharton mentioned this pull request Aug 25, 2016
@athyuttamre
Copy link

+1 this needs to be merged ASAP.

@lfilho
Copy link

lfilho commented Oct 5, 2016

I think this is a sensible addition. Any thoughts, @kadirahq ?

@gkrizek
Copy link

gkrizek commented Oct 7, 2016

+1

@nzwade
Copy link

nzwade commented Oct 8, 2016

+1

@rhyslbw
Copy link

rhyslbw commented Oct 17, 2016

If this isn't being maintained is there a canonical fork we can use?

@jshimko
Copy link

jshimko commented Oct 17, 2016

I don't think this project is really maintained anymore (or at least not often). You're welcome to use my Meteor base image. It's pretty much identical to meteord as far as features.

https://github.com/jshimko/meteor-launchpad

I use and maintain it regularly and I'm always open to contributions.

@rhyslbw
Copy link

rhyslbw commented Oct 17, 2016

Thanks @jshimko, checking it out now

@JulianKingman
Copy link

Thank you @jshimko

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.

8 participants