Make Kernel#method_missing override less naughty #237

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@jwilger
Contributor

jwilger commented Nov 29, 2010

I filed an issue about it. Not sure if there's a way to relate these after the fact. We'll see.

@dchelimsky

This comment has been minimized.

Show comment Hide comment
@dchelimsky

dchelimsky Nov 29, 2010

Member

Unfortunately not - but thanks for the pull request.

Member

dchelimsky commented Nov 29, 2010

Unfortunately not - but thanks for the pull request.

@jwilger

This comment has been minimized.

Show comment Hide comment
@jwilger

jwilger Nov 29, 2010

Contributor

We'll do it backwards, then. Here's the issue. :-)

https://github.com/rspec/rspec-core/issues#issue/236

Contributor

jwilger commented Nov 29, 2010

We'll do it backwards, then. Here's the issue. :-)

https://github.com/rspec/rspec-core/issues#issue/236

@dchelimsky

This comment has been minimized.

Show comment Hide comment
@dchelimsky

dchelimsky Nov 29, 2010

Member

Actually I'd rather use the pull request (which also created issue #237)

Member

dchelimsky commented Nov 29, 2010

Actually I'd rather use the pull request (which also created issue #237)

@jwilger

This comment has been minimized.

Show comment Hide comment
@jwilger

jwilger Nov 29, 2010

Contributor

Fix Kernel#method_missing override so that it gets along with REE 1.8.7-2010.02

Contributor

jwilger commented Nov 29, 2010

Fix Kernel#method_missing override so that it gets along with REE 1.8.7-2010.02

@dchelimsky

This comment has been minimized.

Show comment Hide comment
@dchelimsky

dchelimsky Nov 29, 2010

Member

FYI - I removed _stuff from the method name

Member

dchelimsky commented Nov 29, 2010

FYI - I removed _stuff from the method name

timcharper pushed a commit to timcharper/rspec-core that referenced this pull request Aug 19, 2011

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment