Skip to content

Use RUBY for CLI test subprocesses - #3102

Open
soutaro wants to merge 1 commit into
ruby:masterfrom
Shopify:codex/use-ruby-env-for-cli-tests
Open

Use RUBY for CLI test subprocesses#3102
soutaro wants to merge 1 commit into
ruby:masterfrom
Shopify:codex/use-ruby-env-for-cli-tests

Conversation

@soutaro

@soutaro soutaro commented Aug 26, 2026

Copy link
Copy Markdown
Member

RbConfig.ruby points at Ruby's configured installation path, which may not exist while ruby/ruby tests an uninstalled build. Prefer the RUBY environment variable supplied by its test runner, with RbConfig.ruby as the fallback.

Co-Authored-By: Codex <noreply@openai.com>
soutaro added a commit to soutaro/ruby that referenced this pull request Aug 26, 2026
Co-Authored-By: Codex <noreply@openai.com>
Comment thread test/rbs/cli_test.rb
# That is what happens in ruby/ruby CI, where the tests run with a freshly built `ruby`.
#
BUNDLE_COMMAND = [RbConfig.ruby, "-S", "bundle"]
RUBY_EXECUTABLE = ENV["RUBY"] || RbConfig.ruby

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you update the comment above to explain (what you already have in the PR description)

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.

2 participants