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

Require Active Support rather than Rails #3

Merged
merged 1 commit into from
Feb 26, 2015
Merged

Require Active Support rather than Rails #3

merged 1 commit into from
Feb 26, 2015

Conversation

dylanfareed
Copy link
Contributor

It would be nice to take advantage of hashie_rails without needing to include Rails in a project (in a Grape + Rack app for example). It appears that ActiveSupport is the actual dependency here. This moves Rails to development dependency.

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
s.files = Dir['{app,config,db,lib}/**/*', 'MIT-LICENSE', 'Rakefile', 'README.rdoc']
s.test_files = Dir['test/**/*']

s.add_dependency 'rails', '~> 4.0'
s.add_dependency 'activesupport', '~> 4.2.0'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you wanted to change the dependency to be a 4.2.0, because now anyone on Rails 4.1.x can't use this. No?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah my mistake here. I'll update.

@dblock
Copy link
Collaborator

dblock commented Feb 26, 2015

I also added a CHANGELOG, mind adding a line to it, please?

@Maxim-Filimonov
Copy link
Owner

Hm.. I like the idea but the name of the gem is confusing then :\

@dylanfareed
Copy link
Contributor Author

@Maxim-Filimonov that is indeed a more existential question here...

@Maxim-Filimonov
Copy link
Owner

LGTM! @dblock all good?

@dblock
Copy link
Collaborator

dblock commented Feb 26, 2015

This is good, lets merge. I'll cut a release.

dblock added a commit that referenced this pull request Feb 26, 2015
Require Active Support rather than Rails
@dblock dblock merged commit 75fa78d into Maxim-Filimonov:master Feb 26, 2015
@dblock
Copy link
Collaborator

dblock commented Feb 26, 2015

@Maxim-Filimonov I don't have Rubygems access for this one, could you please run gem owner hashie_rails --add dblock@dblock.org, thanks!

@dylanfareed
Copy link
Contributor Author

Thanks @dblock and @Maxim-Filimonov!

@Maxim-Filimonov
Copy link
Owner

@dblock Ah sorry :\ Just added you.
@dylanfareed thanks for your contribution! 👍

@dblock
Copy link
Collaborator

dblock commented Feb 27, 2015

Released 0.0.2.

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.

3 participants