Skip to content

Conversation

omercnet
Copy link
Contributor

@omercnet omercnet commented Jan 6, 2017

in the effort of motivating people to launch their own instance of freegeoip, this adds the ability to "deploy to heroku" with a single click

@fiorix
Copy link
Owner

fiorix commented Jan 6, 2017

This one is unexpected: https://travis-ci.org/fiorix/freegeoip/jobs/189379740

@fiorix
Copy link
Owner

fiorix commented Jan 6, 2017

Any chance you can use https://github.com/kardianos/govendor instead of godep? I've used both in the past but lately have settled on govendor for a number of reasons.. the first being it doesn't create another directory at the root level of your repo, and places its file under vendor/; also it has a much better cmdline interface.

@omercnet
Copy link
Contributor Author

omercnet commented Jan 6, 2017

yea no problem, godep is default on heroku so I used that
will update soon

Omer Cohen added 2 commits January 6, 2017 14:59
Replacing godep with govendor due to issues with Travis, but also to
reduce package size for those who don't care for heroku and keep the
repo clean
@omercnet
Copy link
Contributor Author

omercnet commented Jan 6, 2017

fixed and rebased with master

@fiorix fiorix merged commit 9f235e4 into fiorix:master Jan 6, 2017
@fiorix
Copy link
Owner

fiorix commented Jan 6, 2017

💯

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.

2 participants