Skip to content

Conversation

@eregon
Copy link
Member

@eregon eregon commented Dec 30, 2025

@eregon eregon force-pushed the method_source_location_from_4_1 branch from 05a208f to f6ab2f9 Compare December 30, 2025 11:36
eregon added a commit to eregon/ruby that referenced this pull request Dec 30, 2025
@eregon
Copy link
Member Author

eregon commented Dec 30, 2025

This doesn't work in CRuby CI, see https://github.com/ruby/ruby/actions/runs/20595820151/job/59150151742?pr=15580
What should I do then?
Exclude the tests with tool/rbs_skip_tests?
Or rebase this on top of 3.10.0 since the failures might be caused by other changes on master?

@eregon
Copy link
Member Author

eregon commented Dec 30, 2025

From #1203 (comment)

Generally, the core library RBS definition targets the latest release of Ruby.

So if that's still the case, and it seems to be according to docs then I should use tool/rbs_skip_tests I guess.
Those docs should mention that then.

BTW these errors mention

💡You can skip this test test_each_paragraph(RBS::AST::Ruby::CommentBlockTest) by adding the name to /home/runner/work/ruby/ruby/src/tool/rbs_skip_tests

But these errors don't.

I also wonder if it's good to run RBS tests related to the core library in ruby/ruby CI since we know at least some of them will fail, so not sure there is much value.
This repo seems to test against ruby-head, not sure how that works though then given those differences.

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