-
Notifications
You must be signed in to change notification settings - Fork 600
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
enable Rack and Puma tests with Ruby 3.3 #2052
Conversation
* test Ruby 3.3 with Puma (via the Rack suite) now that [puma/puma#3165](puma/puma#3165) has been merged * update `add_version` to respect _any_ prefix a human maintainer might want to specify for a version, and not just `=`
resolves #2042
Disregard. We're going to use a SHA and a TODO comment for now, as Puma > v6.3.0 is still nowhere in sight. |
While we continue to wait for a Puma release newer than v6.3.0, let's just go with a SHA for now. That means we need to update our envfile helper to support SHAs. Done.
send 1 argument, not 2
SimpleCov Report
|
@@ -114,8 +108,14 @@ def size | |||
@gemfiles.size | |||
end | |||
|
|||
def add_version(version, twiddle_wakka = true) | |||
def add_version(version) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet edit. Comments were really helpful here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvements!
add_version
to respect any prefix a human maintainer might want to specify for a version, and not just=
add_version
to allow you do whatever Bundler lets you do. You can usegithub
,git
,path
, etc.