-
Notifications
You must be signed in to change notification settings - Fork 39
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
Heroku deploy temporarily broken #457
Comments
@michielbdejong - Using master doesn't actually solve the problem, I think it worked at the time of your message, but once you made the next commit ( |
we do, yes! Although the architecture of the master branch changed considerably and probably we should also fix what |
It definitely looks like there's been some changes. @michielbdejong any thoughts on using yarn instead of npm? Also I'm not sure of the place to leave this, but would you be open to somebody migrating the project from js to typescript? :) |
we did switch to yarn at some point, actually! :) But npm 5 also has a build cache, and there will always be people who want to use npm, so then we switched back to npm. There is a tiny portion of the project already in typescript, namely https://github.com/interledgerjs/ilp-packet. l think there's a lot more other things that could be contributed, which would be higher priority, for instance, we want to switch from using JSON over HTTP to OER over WebSockets, that work is going on in https://github.com/interledgerjs/ilp-plugin-payment-channel-framework/issues - would you be interested in playing a part in those efforts? |
if you want to deploy to Heroku now, use this link: https://dashboard.heroku.com/new?button-url=https%3A%2F%2Fgithub.com%2Finterledgerjs%2Filp-kit&template=https%3A%2F%2Fgithub.com%2Finterledgerjs%2Filp-kit%2Ftree%2Fmaster (notice 'master' instead of 'release' at the end of that URL). It will be fixed again when we tag the next release.
The text was updated successfully, but these errors were encountered: