heroku-buildpack-meteor icon indicating copy to clipboard operation
heroku-buildpack-meteor copied to clipboard

Push rejecting due to bad buildpack(?)

Open jmwierzbicki opened this issue 9 years ago • 4 comments

Aplication was pushed for first time without issues, everything worked fine, but when i made some updates to code, and commited>pushed, following appeared:

remote: Compressing source files... done. remote: Building source: remote: remote: -----> Fetching set buildpack https://github.com/maxko87/heroku-buildpack-meteor.git... done remote: remote: ! Push rejected, failed to detect set buildpack https://github.com/maxko87/heroku-buildpack-meteor.git remote: More info: https://devcenter.heroku.com/articles/buildpacks#detection-failure remote: remote: Verifying deploy... remote: remote: ! Push rejected to myappname. remote: To https://git.heroku.com/myappname.git ! [remote rejected] master -> master (pre-receive hook declined) error: failed to push some refs to 'https://git.heroku.com/myappname.git'

jmwierzbicki avatar Nov 06 '15 23:11 jmwierzbicki

remote: ! Push rejected, failed to detect set buildpack https://github.com/maxko87/heroku-buildpack-meteor.git

This is not the same repo. Maybe open this issue on https://github.com/maxko87/heroku-buildpack-meteor/issues ? I don't know otherwise.

jordansissel avatar Nov 12 '15 18:11 jordansissel

I don't know if this is still an issue, but I had the same error with Jordansissel's builpack. Just for future refference, make sure you do NOT put the .meteor folder in your gitignore. The buildpack needs it to recognize your project as being a meteor project.

Hope this helps someone who runs into the same problem in the future, haven't found this solution anywhere else so far.

("why would you ever put your .meteor folder in the gitignore?" "I don't know.. because I am an idiot..")

TomasDePomas avatar Nov 28 '15 13:11 TomasDePomas

@TomasDePomas, thanks for the tip!

maxko87 avatar Dec 10 '15 10:12 maxko87

@TomasDePomas 's solution worked for me!

martriay avatar Apr 18 '16 22:04 martriay