Skip to content

Small doc improvement #2250

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

Merged
merged 4 commits into from
Jan 4, 2020
Merged

Small doc improvement #2250

merged 4 commits into from
Jan 4, 2020

Conversation

benoittgt
Copy link
Member

I saw missing links or invalid private class information while looking at rspec-rails documentation.

@benoittgt
Copy link
Member Author

+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)

pirj
pirj previously approved these changes Jan 2, 2020
JonRowe
JonRowe previously approved these changes Jan 2, 2020
@benoittgt
Copy link
Member Author

benoittgt commented Jan 2, 2020

Bundler again. (ಠ_ಠ)

rubygems install bundler 2.1.2, then we install bundler 2.1.3 but we choose the wrong version when executing cucumber task.

I was working on a small reproduction script to try to answer to rubygems/bundler#7487 (comment)

At the moment I do

$ rvm use 2.7 && rm -f Gemfile.lock && export RAILS_VERSION='~>6.0' && bundle --binstubs && bundle exec rake smoke:app
# no problem
$ bundle exec rake cucumber
# 💥

And it breaks. I will try to narrow it tomorrow. 💤

@pirj
Copy link
Member

pirj commented Jan 3, 2020

😢

@JonRowe
Copy link
Member

JonRowe commented Jan 3, 2020

Technically, I think its the generated sub apps that are the issue.

@benoittgt
Copy link
Member Author

benoittgt commented Jan 3, 2020

Ok I look a little bit and it seems there is something with unset_bundler_env_vars.

I look a little bit how aruba works but I am not sure I understand how unset_bundler_env_vars works.

With unset_bundler_env_vars commented. bundle exec cucumber returns less errors. But it is not fixed yet. Nevermind CI is green.

If I use the features/support/env.rb patch from #2241 I have even less errors.

Maybe @mvz you have an idea?

EDIT1: CI is green

With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
@benoittgt
Copy link
Member Author

I push this commit for CI testing purpose.

@benoittgt
Copy link
Member Author

Ok it is working... For f1b5011 I simply copy past it. I am not confident to merge without a re-review.

@benoittgt benoittgt requested review from JonRowe and pirj January 3, 2020 22:08
@benoittgt benoittgt dismissed stale reviews from JonRowe and pirj January 3, 2020 22:22

Some code added that need re-review

@pirj
Copy link
Member

pirj commented Jan 3, 2020

@mvz do you mind to approve/criticise this approach?

Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

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

Whatever it takes to get a green build.

in_current_directory do
super(exec_cmd, timeout)
with_unbundled_env do
Copy link
Contributor

@mvz mvz Jan 4, 2020

Choose a reason for hiding this comment

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

This call to with_unbundled_env is probably not needed.

Copy link
Member Author

Choose a reason for hiding this comment

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

So I tried without

And it breaked the build. So I rollback.

Copy link
Contributor

Choose a reason for hiding this comment

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

That's surprising and I'm a little concerned that I don't know what is going on in my own fix, but if it works for now, I guess it's ok. Thanks for checking this, anyway.

Copy link
Member

@JonRowe JonRowe left a comment

Choose a reason for hiding this comment

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

I'm happy if you're happy @benoittgt

@benoittgt
Copy link
Member Author

Thanks for the review you all.

I will be happy if the CI break a little bit less. 😄

Happy new year.

@benoittgt benoittgt merged commit 64b2712 into master Jan 4, 2020
@benoittgt benoittgt deleted the doc-review branch January 4, 2020 16:11
@benoittgt
Copy link
Member Author

I just reran the CI on master to try the last Bundler release 2.1.4. And no issues.

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.

4 participants