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

Remove 186 support #945

Merged
merged 6 commits into from
Jun 14, 2013
Merged

Remove 186 support #945

merged 6 commits into from
Jun 14, 2013

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Jun 13, 2013

More removing 1.8.6 workarounds, as we wont be supporting it in version 3

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling e3732e4 on remove_186_support into b88bcf5 on master.

@myronmarston
Copy link
Member

LGTM. Two further suggestions:

  • Update the gemspec with a line like spec.required_ruby_version = '>= 1.8.7'.
  • Add a commit to the 2-99-maintenance branch that prints a deprecation warning if the user is on 1.8.6 or 1.9.1 so the user knows they can't upgrade.

The gemspec thing would be good on all the gems, but the deprecation warning is sufficient just in rspec-core, I think.

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling e6011c5 on remove_186_support into b88bcf5 on master.

JonRowe added a commit that referenced this pull request Jun 14, 2013
@JonRowe JonRowe merged commit f835cdf into master Jun 14, 2013
@JonRowe JonRowe deleted the remove_186_support branch June 14, 2013 01:21
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.

None yet

3 participants