Skip to content

Conversation

junaruga
Copy link
Member

Add the bundle exec if we can add it to the commands. I didn't add the bundle exec for the rake install, rake test commands, because I saw the following failures when adding the bundle exec to these commands.

Add the `bundle exec` if we can add it to the commands.
I didn't add the `bundle exec` for the `rake install`, `rake test` commands,
because I saw the following failures when adding the `bundle exec` to these commands.

* .github/workflows/ci.yml - step `bundle exec rake install`.
   https://github.com/junaruga/ruby-fiddle/actions/runs/17922685051/job/50961337638
* .github/workflows/libffi.yml - step "Run test" -
  `bundle exec rake test` failed in the `test_no_memory_leak`.
   https://github.com/junaruga/ruby-fiddle/actions/runs/17922685028/job/50961337379
@junaruga
Copy link
Member Author

I see the following failure in my fork repository. Is this failure related to this PR?
https://github.com/junaruga/ruby-fiddle/actions/runs/17923998256/job/50965544249

@junaruga junaruga marked this pull request as ready for review September 22, 2025 18:09
@junaruga
Copy link
Member Author

junaruga commented Sep 22, 2025

This PR comes from the #191 reviews. However, I feel using the bundle exec may not make the tests simple to debug easily.

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.

1 participant