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

Gemspec in 1.3.1 doesn't allow usage with rails 4 #244

Closed
fschwahn opened this issue Mar 21, 2013 · 14 comments · Fixed by #247
Closed

Gemspec in 1.3.1 doesn't allow usage with rails 4 #244

fschwahn opened this issue Mar 21, 2013 · 14 comments · Fixed by #247

Comments

@fschwahn
Copy link

Hi,
I just wanted to update to 1.3.1 to get rid of the ActionController::Integration-deprecation warnings, but it won't work. I see that you added rails as a runtime dependency to the gemspec (which makes sense), but it is defined as

s.add_runtime_dependency('rails', '~> 3.0')

Could you relax it to

s.add_runtime_dependency('rails', '>= 3.0')

or

s.add_runtime_dependency('rails', '>= 3.0', '< 4.1')

or something similar? That would make it possible to use that version with rails 4.

Best regards,
Fabian

@Kosmas
Copy link
Member

Kosmas commented Mar 21, 2013

Thanks Fabian,

@aslakhellesoy @mattwynne should we try to see if it is possible to support both rails 3 and 4 in the same master branch, or make a different branch for rails 4 as suggested before?

@mattwynne
Copy link
Member

On 21 Mar 2013, at 09:25, Kosmas Chatzimichalis [email protected] wrote:

@aslakhellesoy @mattwynne should we try to see if it is possible to support both rails 3 and 4 in the same master branch, or make a different branch for rails 4 as suggested before?

I think it's easier to comprehend if we can use the same code for both versions of Rails.

@alindeman how do you manage this in rspec-rails?

@fschwahn
Copy link
Author

I'm already using cucumber-rails 1.3.0 with rails 4 and haven't run into any issues so far, besides the deprecation warnings. I'm using database_cleaner 1.0.0.RC1, which claims to be compatible with rails 4.
It is only a small project, but it seems capybara-rails is already somewhat compatible with rails 4.

@alindeman
Copy link
Contributor

@Kosmas
Copy link
Member

Kosmas commented Mar 21, 2013

I just tried to put the dependency to rails >= 3.0 and use the appraisal with rails 4 gems but 17 out of the 20 tests fail. Will have a closer look.

@alindeman thanks. Is there a reason that you have activesupport,actonpack and railties separated instead of rails?

@localhots
Copy link

This is an issue for me also, it was working fine with rails 4 and cucumber-rails 1.3.0

@Kosmas
Copy link
Member

Kosmas commented Apr 23, 2013

There is a branch called master_rails4_test that is set up to make the current master version work with rails 4 and rails 3, by using the appraisal created gemfiles.
At the moment most of the tests are passing (some of them are commented out), for the rails 3 but failing for rails 4.
The plan is to make the tests pass for all versions in that branch and if everything works to merge it back to the master branch.

Anyone wishing to help with this please make a pull request in that branch (master_rails4_test).

@miketheman
Copy link

rails 4 has been released, any news on merging this branch to master and releasing a gem compatible with both?

@Kosmas
Copy link
Member

Kosmas commented Jul 5, 2013

Hi @miketheman there is a discussion here about the current issues to get the gem working with rails 4:

https://groups.google.com/forum/?fromgroups=#!topic/cukes-devs/FGXiTvf1vkg

@miketheman
Copy link

@Kosmas thanks for the link. From the thread, it looks like thing are going ok?

@greg606
Copy link

greg606 commented Jul 19, 2013

Any progress?

@Kosmas
Copy link
Member

Kosmas commented Jul 19, 2013

Hi @greg606 the progress is in the latest travis build here: https://travis-ci.org/cucumber/cucumber-rails

As you can see the rails 4 tests are failing.

@scriptfans
Copy link

Very concerned about it,Looking forward someone can solve this problem.

@Kosmas
Copy link
Member

Kosmas commented Aug 23, 2013

version 1.4.0 is rails 4 compatible now.

@Kosmas Kosmas closed this as completed Aug 23, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants