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

Support Rails 4.x and fix generating examples #120

Merged
merged 16 commits into from
Feb 12, 2015

Conversation

a2ikm
Copy link
Contributor

@a2ikm a2ikm commented Feb 11, 2015

I tweaked tests around setup and teardown, but not changed essentially.

I removed Gemfile.lock from repository because it can change for ActiveSupport's dependency on Test::Unit or minitest and CRuby's version. But it doesn't make any wrong effects on developing and releasing.

has_many :subscribed_gems, :through => :subscriptions,
:source => :rubygem,
:order => "name ASC"
end
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is the sort of version switching I was dreading having to do... well done, thank you :)

@kerrizor
Copy link
Collaborator

This is fabulous thank you! ❤️

@a2ikm
Copy link
Contributor Author

a2ikm commented Feb 11, 2015

@kerrizor I updated about switching in 6e3fd70. Should I squash it?

@kerrizor
Copy link
Collaborator

In this case, yes please :)

@a2ikm
Copy link
Contributor Author

a2ikm commented Feb 12, 2015

I squashed commits.
@kerrizor Could you check that?

kerrizor added a commit that referenced this pull request Feb 12, 2015
Support Rails 4.x and fix generating examples
@kerrizor kerrizor merged commit 60054e5 into voormedia:master Feb 12, 2015
@kerrizor
Copy link
Collaborator

Looks good from here! Merging and will cut a new release shortly.

Thanks again for the hard work here!

@a2ikm
Copy link
Contributor Author

a2ikm commented Feb 13, 2015

Thank you for merging!
I'm happy to use this gem with rails4!

@a2ikm a2ikm deleted the support_rails_4.x branch February 13, 2015 04:44
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

2 participants