-
Notifications
You must be signed in to change notification settings - Fork 41
Use ruby/actions workflow for ruby versions #136
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
base: master
Are you sure you want to change the base?
Use ruby/actions workflow for ruby versions #136
Conversation
.github/workflows/ci.yml
Outdated
with: | ||
engine: cruby | ||
min_version: 2.5 | ||
versions: "[debug]" |
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.
Perhaps like this:
versions: "[debug]" | |
versions: | |
- debug |
...or perhaps it's called head
, not debug?
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.
Thank you!
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.
It seems that we can't use JSON array here...: https://github.com/ruby/fiddle/actions/runs/7472168103/workflow
The workflow is not valid. .github/workflows/ci.yml (Line: 14, Col: 9): A sequence was not expected
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.
thanks. fixed 6e2f4b9
Co-authored-by: Olle Jonsson <olle.jonsson@gmail.com>
We may need to add a condition for fiddle/.github/workflows/ci.yml Line 51 in fd4bb4e
BTW, do we need both of |
Could you rebase on master? |
this PR has been changed to use the ruby/actions workflow for ruby versions.