-
Notifications
You must be signed in to change notification settings - Fork 2k
Deploying meanJS to Heroku fails #1928
Comments
Try replacing node engine version ">=6.10.0" with "9.2.0". |
@Ghalleb: How would I change it, or where exactly? I'm just clicking the button in the main ReadME: Thanks... |
In fact you have to change the code, in the package.json file.
I created this button. It was working okay untill heroku changed node.js
version. Let me some time and I will publish a fix.
In the meantime, you could fork the repo, change the code as I said and it
will work as expected.
Le 29 nov. 2017 14 h 31, "dylib" <[email protected]> a écrit :
… @Ghalleb <https://github.com/ghalleb>: How would I change it, or where
exactly? I'm just clicking the button in the main ReadME:
[image: button] <https://heroku.com/deploy>
Thanks...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1928 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AClWiYDlMBCgMa1tibqsitnEdON9qrhYks5s7QhIgaJpZM4QskX9>
.
|
@Ghalleb: Ah that makes sense. Thanks for the tips on how to go about updating it... I'm not in a huge hurry though, so I might just wait for you to publish the fix. :) |
@dylib @simison @mleanos @lirantal |
@Ghalleb: The new version works great! There were just a couple warnings in the build log:
Deploying to Heroku went smooth, so thanks very much for updating it! |
When attempting the Heroku deploy method the build fails:
Any suggestions? Thank you.
* Note this also happens with meanTorrent, as noted with issue #36 here.
The text was updated successfully, but these errors were encountered: