Skip to content

Conversation

@nobu
Copy link
Member

@nobu nobu commented Jul 17, 2025

In ruby/ruby test-bundled-gems, rbs has many failures on Windows. Currently we ignore all failures in this test totally, but that is the test-run makes no sense but just wastes times.
So I want to also specify another file for the tests to be skipped only on Windows, in addition to the platform independent skipping list file, that we are using already.

@nobu nobu force-pushed the skip_tests-files branch from e5e0bb2 to a6bc4dd Compare July 17, 2025 02:57
@soutaro
Copy link
Member

soutaro commented Jul 17, 2025

The master branch is unstable, so I don’t think it makes sense to send it to the Ruby repo.

I think opening a PR for aaa-3.9.x branch and testing it on Ruby CI with the commit would be fine.

In ruby/ruby test-bundled-gems, rbs has many failures on Windows.
Currently we ignore all failures in this test totally, but that is the
test-run makes no sense but just wastes times.  So I want to also
specify another file for the tests to be skipped only on Windows, in
addition to the platform independent skipping list file, that we are
using already.
@nobu nobu force-pushed the skip_tests-files branch from a6bc4dd to dbdc766 Compare July 17, 2025 06:06
Copy link
Member

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

Thanks! 🙏

@soutaro soutaro added this pull request to the merge queue Aug 14, 2025
Merged via the queue into ruby:master with commit 607dc05 Aug 14, 2025
20 checks passed
soutaro added a commit that referenced this pull request Aug 14, 2025
Allows the use of a path list as RBS_SKIP_TESTS
@soutaro soutaro added this to the RBS 4.0 milestone Sep 8, 2025
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