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

CI against Ruby 2.6.0 #1338

Closed
wants to merge 3 commits into from
Closed

Conversation

koic
Copy link
Member

@koic koic commented Dec 30, 2018

Summary

CI against Ruby 2.6.0.

Details

Ruby 2.6.0 has been released and this Ruby version is available on CircleCI.

Checklist:

  • I've added tests for my code
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@ghost ghost assigned koic Dec 30, 2018
Copy link
Contributor

@olleolleolle olleolleolle left a comment

Choose a reason for hiding this comment

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

You need to add the 2.6 job to the "workflows/jobs" list at the end of the config.yml.

It does not run, yet.

@koic koic force-pushed the ci_against_ruby_2_6_0 branch from 349a185 to 62d131f Compare December 30, 2018 18:40
@koic
Copy link
Member Author

koic commented Dec 30, 2018

Oops! I updated this PR.

@luke-hill
Copy link
Contributor

Are these failing because of the other PR around BigDecimal.new? Or something else?

Have you had time to triage @koic ?

@olleolleolle
Copy link
Contributor

This build error:

An error occurred while loading ./spec/cucumber/cli/configuration_spec.rb.
Failure/Error: require 'gherkin/gherkin'

LoadError:
  cannot load such file -- google/protobuf_c

Seems like all of them are about that.

@danascheider
Copy link
Contributor

Looks to be related to an issue with the google-protobuf gem protocolbuffers/protobuf#5161

  - and it requires special installation from protocolbuffers/protobuf#5161
@ghost ghost assigned olleolleolle Jan 1, 2019
@olleolleolle
Copy link
Contributor

olleolleolle commented Jan 1, 2019

In order to move forward, I made an experimental addition to this PR: run the gem install as described in the Description of protocolbuffers/protobuf#5161

That did not work, yet. I'm sorry about moving too quickly.

@olleolleolle
Copy link
Contributor

The protobuf Issue has been closed fixed in the other repo, but there's no new gem release yet.

https://rubygems.org/gems/google-protobuf

@luke-hill
Copy link
Contributor

@koic If you add a new gem requirement of: gem 'google-protobuf', '~> 3.7.0.rc.2' this should in theory start passing.

If you've got time do you want to give that a go?

@koic
Copy link
Member Author

koic commented Mar 7, 2019

Yep! I will try it.

@koic
Copy link
Member Author

koic commented Mar 7, 2019

Unfortunately the following error occurred when running bundle update.

% bundle update google-protobuf
Fetching gem metadata from https://rubygems.org/.........
Resolving dependencies...
Bundler could not find compatible versions for gem "google-protobuf":
  In Gemfile:
    google-protobuf (~> 3.7.0.rc.2)

    cucumber was resolved to 4.0.0.rc.1, which depends on
      gherkin (~> 6.0) was resolved to 6.0.15, which depends on
        cucumber-messages (~> 2.0.0) was resolved to 2.0.0, which depends on
          google-protobuf (= 3.6.1)

First of all, the dependency needs to be resolved.

cucumber-formatter-dots 1.1.2 (latest) requires cucumber-messages ~> 2.0.0.
https://rubygems.org/gems/cucumber-formatter-dots/versions/1.1.2

cucumber-messages 2.0.0 requires google-protobuf = 3.6.1.
https://rubygems.org/gems/cucumber-messages/versions/2.0.0

Each version needs to be bumped. I will open PR to cucumber-messages repo and cucumber-formatter-dots repo for resolving dependencies.

@koic
Copy link
Member Author

koic commented Mar 7, 2019

The cucumber-messages gem release is required to resolve dependencies of cucumber-messages to google-protobuf. It has already been fixed in the cucumber-messages repo code base.
cucumber/common#571 (comment)

@koic
Copy link
Member Author

koic commented Mar 7, 2019

Also cucumber-formatter-dots gem will wait for release as well.

@luke-hill
Copy link
Contributor

The messages fix has been applied. This should automatically pull through

@aslakhellesoy
Copy link
Contributor

Superseded by #1346

@koic koic deleted the ci_against_ruby_2_6_0 branch April 9, 2019 05:09
@lock
Copy link

lock bot commented Apr 15, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants