Skip to content

Conversation

@makenowjust
Copy link
Contributor

Fix #577

It seems `eval(..., __LINE__ + 1)` does not work.
Other similar tests are also pended on TruffleRuby, so I think it
is acceptable.
Copy link
Member

@tompng tompng left a comment

Choose a reason for hiding this comment

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

Looks good, thank you 👍

@tompng tompng merged commit fd49135 into ruby:master May 23, 2023
matzbot pushed a commit to ruby/ruby that referenced this pull request May 23, 2023
(ruby/irb#589)

* Allow `show_source` for private methods

Fix ruby/irb#577

* Pend tests on TruffleRuby

It seems `eval(..., __LINE__ + 1)` does not work.
Other similar tests are also pended on TruffleRuby, so I think it
is acceptable.

* Use `private_method_defined?` instead of `defined?`
@st0012 st0012 added the enhancement New feature or request label May 23, 2023
@makenowjust makenowjust deleted the private-show-source branch May 24, 2023 01:17
@st0012 st0012 mentioned this pull request Jun 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ShowSource of private method

3 participants