-
Notifications
You must be signed in to change notification settings - Fork 89
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
Ruby::MethodDefinitionMissing with attr_reader #1036
Comments
Attempted a fix in #1037, probably not a very good one. |
Apparently the solution (as of 1.6.0) is to use
Which feels very redundant given that RBS's |
lloeki
added a commit
to DataDog/graft-rb
that referenced
this issue
Jun 4, 2024
Two main cases: - `attr_*` needs `@dynamic`: soutaro/steep#1036 - three `define_method` because of the `RUBY_VERSION` condition
lloeki
added a commit
to DataDog/graft-rb
that referenced
this issue
Jul 3, 2024
Two main cases: - `attr_*` needs `@dynamic`: soutaro/steep#1036 - three `define_method` because of the `RUBY_VERSION` condition
lloeki
added a commit
to DataDog/graft-rb
that referenced
this issue
Jul 10, 2024
Two main cases: - `attr_*` needs `@dynamic`: soutaro/steep#1036 - three `define_method` because of the `RUBY_VERSION` condition
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
attr_reader foo: untyped
in RBS defines the signature for the method#foo
as woulddef foo: () -> untyped
Steep::Diagnostic::Ruby::MethodDefinitionMissing
Steep expects the method#foo
from the aboveattr_reader
in Ruby defines the method#foo
yet Steep can't find it, even though both.rb
and .rbsuse
attr_reader` semanticsExpected result:
attr_reader foo: untyped
should be able to findattr_reader
-defined methodsdef foo: () -> untyped
+@foo: untyped
should also work with and findattr_reader :foo
attr_writer
andattr_accessor
and#foo=
Actual result:
The text was updated successfully, but these errors were encountered: