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

Is this necessary with Meteor 1.4 anymore? #225

Open
jedwards1211 opened this issue Sep 2, 2016 · 4 comments
Open

Is this necessary with Meteor 1.4 anymore? #225

jedwards1211 opened this issue Sep 2, 2016 · 4 comments

Comments

@jedwards1211
Copy link

It looks like the code in Master is basically just calling meteor build and tweaking its package.json a bit.

I thought it was adding package.json entries for everything in programs/server/npm but it looks like that's not the case.

meteor build is sufficient for 1.4 projects right?

@theworkflow
Copy link
Contributor

IMO it's totally up to the end user to decide if this is still needed. I think so because while it does look like it's only setting up the package.json, it's ensuring that engines are set up and easy to deploy on any infrastructure. While that's the goal, the meteor build command doesn't really do that. On top of that, it allows for custom JSON to be added to the package.json. There are some great things that this module does in the long run.

@jedwards1211
Copy link
Author

I see...I'm guessing that meteor build has gotten closer to outputting a normal node project over the years though?

@jedwards1211
Copy link
Author

It looks like back in the original version it was actually doing a lot more work, like building the list of dependencies manually.

@ramijarrar
Copy link

ramijarrar commented Sep 7, 2016

We dropped it while waiting for 1.4 compatibility & taking a closer look at what it was doing relative to the overhead. It only took us a few hours to write a simple script which makes the same modifications to package.json without any of the dependencies.

While I think there is still a place for a tool like this post-build, the way it's written at the moment (which probably dates back to when it was performing more complex tasks) doesn't justify the value it is providing.

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

No branches or pull requests

3 participants