Skip to content
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

Use hubot style app deployment #18

Closed
wants to merge 1 commit into from
Closed

Use hubot style app deployment #18

wants to merge 1 commit into from

Conversation

camshaft
Copy link
Contributor

@camshaft camshaft commented Feb 5, 2013

This allows you to setup a project to depend on strider instead of cloning the strider repo and maintain the upstream changes that occur. A strider instance would use a setup similar to this template and that is what gets deployed to heroku. That way, I can version my dependency on strider and get any updates that occur just by changing my package.json. This is very similar to the way github's hubot works.

@niallo
Copy link
Member

niallo commented Feb 7, 2013

I think the idea is a good one. Thanks for making the pull request.

Some of the details in the diff I don't follow (for example the config.js changes which seem to duplicate what everypaas already does) and I don't think I can accept the package.json changes.

That being said I like the idea of being able to depend on strider and version it.

@camshaft
Copy link
Contributor Author

camshaft commented Feb 7, 2013

Could you comment on the particular lines and we can talk through it :)

@camshaft
Copy link
Contributor Author

camshaft commented Feb 7, 2013

Ok. I've got some work to do 😄 I'm glad you like it though. I'll start on it tomorrow and push my changes when I'm done. Thanks for your work on strider!

@niallo
Copy link
Member

niallo commented Feb 7, 2013

Awesome. Thanks for hacking on this. BTW you might be interested in this: https://github.com/niallo/deadlift

Right now it is customized for Strider, but I have been meaning to make it more generic in future.

@camshaft
Copy link
Contributor Author

camshaft commented Feb 7, 2013

I actually tried it a couple of weeks ago and didn't work for me. If I remember correctly I think it just got stuck on one of the steps and so I just did my own deployment. But I really like the idea and I'm sure there's been changes made since then.

@peterbraden
Copy link
Contributor

In terms of the config stuff, I'm attacking that in the removeConfig branch

@camshaft
Copy link
Contributor Author

@peterbraden awesome! I was going to work on this stuff this weekend. Sorry it's taking so long :(

@niallo
Copy link
Member

niallo commented Mar 21, 2013

This has been solved by merging @peterbraden's removeConfig branch and by #30. See sample usage in README.md.

@niallo niallo closed this Mar 21, 2013
@camshaft
Copy link
Contributor Author

@niallo you're awesome! thanks for taking care of this :)

@patcon patcon mentioned this pull request Jan 7, 2014
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.

None yet

3 participants