-
Notifications
You must be signed in to change notification settings - Fork 4
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
updating crichton reference #29
Conversation
@@ -3,7 +3,7 @@ source 'https://rubygems.org' | |||
# Bundle edge Rails instead: gem 'rails', github: 'rails/rails' | |||
gem 'rails', '3.2.0' | |||
|
|||
gem 'crichton', git: 'git@github.com:mdsol/crichton.git', tag: '0.0.3' | |||
gem 'crichton', git: 'git@github.com:brutski/crichton.git', tag: '0.0.3' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brutski I forked current mdsol-share repo here: https://github.com/fosrias/crichton. What do you think about keeping this all in this repo? If so, how about updating the gemfile and maybe fixing the links in the README of crichton pointing here (or I can). Thanks for this and sorry I am slow to this. Tied up at RESTfest. |
@brutski Fixed docs myself: https://github.com/fosrias/crichton/pulls/1 |
@fosrias so I looked at your branch of crichton. Exactly what I was talking today: don't really like the latest commit, which adds |
So, if things work should we leave it for now. They may change this and it would make it easier to pull over new releases if they happen. I agree it is less than optimal. Thoughts? On Oct 1, 2014, at 7:25 PM, brutski notifications@github.com wrote:
|
@fosrias crichton-demo-service works. updated PR to point to git@github.com:fosrias/crichton.git; v.0.0.3 |
@brutski Thx. Merging. |
@fosrias please merge. Updating Gemfile to point to the latest crichton making crichton-demo-service work again.