Skip to content

Update Heroku instructions to include buildpack sequence #1283

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

Closed
wants to merge 1 commit into from

Conversation

dpaola2
Copy link

@dpaola2 dpaola2 commented Feb 22, 2018

No description provided.

@ytbryan
Copy link
Contributor

ytbryan commented Feb 22, 2018

Is this necessary? Heroku seems to have it installed by default, correct me if i'm wrong.

@dpaola2
Copy link
Author

dpaola2 commented Feb 22, 2018

I was following the instructions on a (nearly) vanilla rails app and ran into #1282 . This was my issue. Just trying make sure others behind me don't hit the same (extremely frustrating) obstacle.

@p0wl
Copy link
Contributor

p0wl commented Feb 22, 2018

We had to do this by hand as well.

@matthewd
Copy link
Member

cc @hone

@joecorkerton
Copy link

I ran into the same problem, thanks for the fix. I also created an issue in the ruby buildpack repo as I believe that is where this should be fixed.

heroku/heroku-buildpack-ruby#712

@sunnyrjuneja
Copy link
Contributor

We also had to do this. It's also helpful to specify the version of node / yarn in package.json. I think it's worth mentioning in the PR.

@guilleiguaran
Copy link
Member

Looks like this has been fixed already on the Heroku buildpack.

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.

7 participants