Skip to content

Bundle rbs-3.4.3 #9702

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

Merged
merged 3 commits into from
Jan 25, 2024
Merged

Bundle rbs-3.4.3 #9702

merged 3 commits into from
Jan 25, 2024

Conversation

soutaro
Copy link
Member

@soutaro soutaro commented Jan 25, 2024

Updating bundled rbs to 3.4.3 introduced a test failure.

  • test_loading_from_rbs_collection__gem_version_mismatch(RBS::EnvironmentLoaderTest) is failing because it requires rbs-amber gem, which is generated during tests into Bundler context. Since the test is running without Bundler, it should be skipped. (This is a test case added in rbs-3.4.3, so it's a bug of rbs-3.4.3...)
  • Another failure is test_argument_forwarding(RBS::RbPrototypeTest), which is introduced during recent parser changes. So, I'm just moving the ignore from TEST_BUNDLED_GEMS_ALLOW_FAILURES to more fine grained rbs_skip_tests. We will enable the tests before Ruby 3.4 release, once the parser got stable enough.

soutaro and others added 2 commits January 25, 2024 22:26
Allow typeprof test failures
@soutaro soutaro enabled auto-merge (squash) January 25, 2024 13:28
@soutaro soutaro disabled auto-merge January 25, 2024 13:28
@soutaro
Copy link
Member Author

soutaro commented Jan 25, 2024

typeprof is still failing, but it's ignored:

RuntimeError: Unknown insn: #<struct TypeProf::ISeq::Insn insn=:pushtoarray, operands=[], lineno=5, code_range=(5,4)-(5,27), definitions=nil>

It looks like unrelated to rbs update. So leaving it for @mame. 🙏

@soutaro soutaro merged commit b822f9f into ruby:master Jan 25, 2024
@soutaro soutaro deleted the soutaro-patch-2 branch January 25, 2024 23:47
@soutaro soutaro mentioned this pull request Jan 29, 2024
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