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

Fix for rspec 3.3 #27

Closed
wants to merge 4 commits into from
Closed

Fix for rspec 3.3 #27

wants to merge 4 commits into from

Conversation

eitoball
Copy link
Contributor

This PR should fix issue that is discussed in #25. As @myronmarston mentioned:

In general, we prefer if extensions limit themselves to our public APIs, as that prevents this kind of issue. What kind of public APIs would you want to no longer have to monkey patch RSpec?

We need to this about public APIs and provide some inputs to rspec project.

@zephiransas
Copy link

This PR fix the problem for me 👍

@mattheworiordan
Copy link

👍 I have this issue

@cciollaro
Copy link

👍 works for me

@galulex
Copy link

galulex commented Jul 6, 2015

👍

@michaelglass
Copy link
Contributor

will review this in the next 24 hours, thanks!

@matthieua
Copy link

Thanks @michaelglass!

@michaelglass
Copy link
Contributor

I'm going to split this up into a few different PRs and get it merged...

This was referenced Jul 9, 2015
@michaelglass
Copy link
Contributor

moved to #32

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.

7 participants