-
Notifications
You must be signed in to change notification settings - Fork 9
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
Bump rack from 2.2.9 to 3.1.7 #541
Conversation
|
There's a Trello card for this |
b987b8e
to
2ce5301
Compare
Bumps [rack](https://github.com/rack/rack) from 2.2.9 to 3.1.7. - [Release notes](https://github.com/rack/rack/releases) - [Changelog](https://github.com/rack/rack/blob/main/CHANGELOG.md) - [Commits](rack/rack@v2.2.9...v3.1.7) --- updated-dependencies: - dependency-name: rack dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
2ce5301
to
538b0f9
Compare
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.
Could it be worth linking to the commit where this behaviour changed in rack?
This now returns an instance of `PrometheusExporter::Middleware` directly, instead of inside an array. From a quick scan, I couldn't find mention of this in the [Rack 3 upgrade guide][1], but the URL below takes you to the commit and lines in which this change was made rack/rack@b95cb72#diff-79892c6315a7f9c02489803e95ce828a8a32759b9f8f196aa98e6826eb562e55L69-R67 [1]: https://github.com/rack/rack/blob/main/UPGRADE-GUIDE.md
5616fb1
to
b0ca019
Compare
Nice find - couldn't spot it in the upgrade guide! I've added a link to those lines |
Trello
Bumps rack from 2.2.9 to 3.1.7.
Release notes
Sourced from rack's releases.
Changelog
Sourced from rack's changelog.
... (truncated)
Commits
4bb2f72
Bump patch version.1c1e413
Ignore external tests directory.b4a1036
Prepare for 3.1.7 release.d0da91b
Add more external tests.f6f1510
ImproveRack::Response
content-length header generation. (#2219)fb339e0
Fix encoding setting for non-binary IO-like objects in MockRequest#env_fore21872d
Do not remove escaped opening/closing quotes for content-disposition filenames5c3d79f
Synchronize changelog with HEAD.bca33b4
Allow empty PATH_INFO. (#2214) (#2234)98aa947
Bump patch version.Most Recent Ignore Conditions Applied to This Pull Request
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)