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
Rails 4 support #36
Rails 4 support #36
Conversation
…use correct require for this newer version.
…t and Rails 3 don't care.
@@ -0,0 +1,7 @@ | |||
language: ruby | |||
rvm: | |||
- 1.9.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.
Is good to add 1.8.7 in the build matrix only to 3.2.x
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.
Add 1.8.7 to rvm
and adding this should do it:
matrix:
exclude:
- rvm: 1.8.7
gemfile: Gemfile
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.
Done in 08b092f
Thank you so much |
matrix: | ||
exclude: | ||
- rvm: 1.8.7 | ||
gemfile: gemfiles/Gemfile.rails-3.2.x |
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.
Hey guys, I did a mistake here. It should be
matrix:
exclude:
- rvm: 1.8.7
gemfile: Gemfile
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.
Ah, nevermind. It has beed fixed already.
Hi guys,
On the way to make InheritedResources and its dependencies Rails 4 compatible, please review this pull request.
In this pull request I was trying to take into account all the comments I got in heartcombo/responders#60
Please let me know if you have any comments, suggestions, etc.
Thank you.