Skip to content
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

add Kernel#respond_to_missing? #1518

Merged
merged 3 commits into from
Sep 19, 2023

Conversation

ParadoxV5
Copy link
Contributor

@ParadoxV5 ParadoxV5 commented Sep 15, 2023

rbs/test/test_helper.rb

Lines 112 to 113 in fce99b9

private
def respond_to_missing?: (Symbol, bool) -> bool

My current project overrides respond_to_missing? and fails Steep on the courtesy super calls.

This originally based on #1504 because they conflict (this’d be Object#respond_to_missing? otherwise). Since the prerequisite is now (forced-pushed and) merged, this PR now bases on master itself.

Copy link
Contributor Author

@ParadoxV5 ParadoxV5 left a comment

Choose a reason for hiding this comment

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

I wrote my first RBS test! 😄

Comment on lines 897 to 898
"(Symbol, bool) -> bool",
obj, :respond_to_missing?, :to_s, true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see other (bool) tests simply use bool for the type and true for the arg, instead of separate tests for true and false. I suppose I’ll match.

test/stdlib/Kernel_test.rb Outdated Show resolved Hide resolved
@ParadoxV5 ParadoxV5 marked this pull request as ready for review September 17, 2023 04:11
@soutaro soutaro force-pushed the Kernel_i_respond_to_missing branch from 7dab7f4 to 3502ed6 Compare September 19, 2023 08:25
@soutaro soutaro enabled auto-merge September 19, 2023 08:26
@soutaro soutaro added this to the RBS 3.3 milestone Sep 19, 2023
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 Sep 19, 2023
Merged via the queue into ruby:master with commit 9a5765e Sep 19, 2023
@ParadoxV5 ParadoxV5 deleted the Kernel_i_respond_to_missing branch September 21, 2023 21:40
@soutaro soutaro added the Released PRs already included in the released version label Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Released PRs already included in the released version
Development

Successfully merging this pull request may close these issues.

2 participants