-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add Truffleruby support #1612
Add Truffleruby support #1612
Conversation
Their most recently public accepted PR is: digital-fabric/extralite#5 |
Out of curiosity: why did you extract rubocop from the default rake tasks? I think it is a good idea indeed. But will that have any impact on the Truffleruby support? |
The main reason is that currently we have memory issue with GithubActions and I noticed that running tests together with rubocop fails the pipeline, which is currently the team is investigating, but I didn't want to block this PR with that |
another example of the memory related issue what team is currently investigating: oracle/truffleruby#2591 |
Thanks for the explanation 👍 |
Add Truffleruby support to CI
@aslakhellesoy @aurelien-reeves is there anything else I can do here? |
Looks pretty good actually, thank you for your contribution. Would you have the possibility to update the README to add truffleruby in the supported platforms? And the CHANGELOG too? |
@aurelien-reeves done |
oO I missed that truffleruby is passing all cucumber tests now, that's great :) |
@eregon thanks for the review! |
Hi @gogainda, Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾 In return for this generous offer we hope you will:
On behalf of the Cucumber core team, |
@gogainda the truffleruby build is failing since a few days: https://github.com/cucumber/cucumber-ruby/actions |
The error is:
I'll take a look (cc @bjfish). |
This seems already fixed, I think we just need to wait the next run. |
Thanks @eregon :) |
Indeed, it's green now: https://github.com/cucumber/cucumber-ruby/runs/6382227723?check_suite_focus=true |
Great, thanks again @eregon Regarding truffleruby, it seems that at the moment, the CI is based on the development head. |
Yes, feel free to try Testing against Testing both also makes sense, then it's easy to differentiate if some change in cucumber/deps broke on TruffleRuby in general or if it's due to some change in truffleruby-head. |
Noted! Thanks for the info! :) |
Add Truffleruby support to CI
Extract Rubocop linting into separate workflow