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

Add support for Ruby 3.0.0 #46

Merged
merged 1 commit into from
May 21, 2021
Merged

Add support for Ruby 3.0.0 #46

merged 1 commit into from
May 21, 2021

Conversation

pblesi
Copy link
Contributor

@pblesi pblesi commented Apr 23, 2021

This addresses issue: #40

In addition to adding support for Ruby 3.0.0, this commit does the
following:

  • Adds support for Ruby 2.7.2
  • Removes support for Ruby 2.2.10
  • Adds support for Bundler < 3.0
  • Drops support for Bundler < 2.2.0
  • Adds a ci option to the project generator

@pblesi pblesi force-pushed the master branch 2 times, most recently from 79b6db7 to a2c1ebf Compare April 24, 2021 17:21
@coopergillan
Copy link
Contributor

coopergillan commented May 13, 2021

@pblesi - this one now needs a merge conflict resolution due to gem upgrades.

@ClashTheBunny ClashTheBunny mentioned this pull request May 14, 2021
Copy link

@scannillo scannillo left a comment

Choose a reason for hiding this comment

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

👋 Thanks for making this update. The Braintree SDK team uses Runbook (via gist) to release our Ruby SDK, which recently bumps to support 2.6+. So this PR should also fix a current issue with our release script.

If you need any additional 👀 or help on it, let us know!

@pblesi pblesi force-pushed the master branch 2 times, most recently from 39645ed to ed5d43e Compare May 19, 2021 04:00
Copy link

@ClashTheBunny ClashTheBunny left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link

@hollabaq86 hollabaq86 left a comment

Choose a reason for hiding this comment

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

@pblesi ❤️ this work updating runbook. I left a few questions/comments around versioned testing and dependencies.

- 2.4.6
- 2.5.5
- 2.6.3
- 2.4.10

Choose a reason for hiding this comment

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

the Ruby core community dropped support for versions below 2.6, I'm thinking we should drop any versions below 2.6, as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would prefer to continue to support these versions as long as it is not inconvenient. On the off chance that someone needs a feature from this version, but they find it difficult to upgrade their Ruby version, I want to be accommodating to their needs.

Choose a reason for hiding this comment

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

Debian oldstable, still in lts, is ruby 2.3.3.

Choose a reason for hiding this comment

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

wowwwwwwwww

runbook.gemspec Outdated
@@ -42,7 +43,7 @@ Gem::Specification.new do |spec|

spec.add_development_dependency "appraisal", "~> 2.2"
spec.add_development_dependency "aruba", "~> 0.14"
spec.add_development_dependency "bundler", "~> 1.15"
spec.add_development_dependency "bundler", ">= 2.2.0", "< 3.0"

Choose a reason for hiding this comment

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

why not use ~> 2.2 here, instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that makes sense to me. I can't recall why I expressed it this way. Will update.

In addition to adding support for Ruby 3.0.0, this commit also:

* Adds support for Ruby 2.7.2
* Removes support for Ruby 2.2.10
* Adds support for Bundler < 3.0
* Drops support for Bundler < 2.2.0
* Adds a ci option to the project generator
@coopergillan coopergillan merged commit 261f82d into braintree:master May 21, 2021
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.

5 participants