-
Notifications
You must be signed in to change notification settings - Fork 2k
chore(deps): bump body-parser from 1.20.3 to 2.2.0 #4383
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
Conversation
Bumps [body-parser](https://github.com/expressjs/body-parser) from 1.20.3 to 2.2.0. - [Release notes](https://github.com/expressjs/body-parser/releases) - [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md) - [Commits](expressjs/body-parser@1.20.3...v2.2.0) --- updated-dependencies: - dependency-name: body-parser dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the WalkthroughA single dependency update bumps Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
Comment |
a104db3 to
c41236f
Compare
…and_yarn/body-parser-2.2.0
c41236f to
3d4e6ea
Compare
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
packages/bruno-tests/package.json (1)
22-22: Use consistent version pinning strategy.The
body-parserdependency is pinned at2.2.0while all other dependencies use caret ranges (e.g.,^1.8.3,^4.21.2). Consider using^2.2.0for consistency unless pinning is intentional for test stability.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
packages/bruno-tests/package.json(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
- GitHub Check: Playwright E2E Tests
- GitHub Check: Unit Tests
- GitHub Check: SSL Tests - Windows
- GitHub Check: SSL Tests - macOS
- GitHub Check: SSL Tests - Linux
- GitHub Check: CLI Tests
🔇 Additional comments (1)
packages/bruno-tests/package.json (1)
22-22: Verify Node.js and express compatibility with body-parser 2.2.0.The bump from
1.20.3to2.2.0is a major version upgrade. Per the PR notes, body-parser v2.0.0 requires Node.js 18+ as the minimum. Confirm:
- Project's minimum Node.js version is 18 or higher (check
package.json,.nvmrc, CI workflows, or engine field).- Express
4.21.2(already in dependencies) is compatible with body-parser2.2.0.- Test suite passes with the updated version (no API breaking changes affecting test code).
Bumps body-parser from 1.20.3 to 2.2.0.
Release notes
Sourced from body-parser's releases.
... (truncated)
Changelog
Sourced from body-parser's changelog.
... (truncated)
Commits
0aa4e112.2.0 (#597)4d85c4crefactor: cleanup parser options (#596)d11899brefactor: normalize common options for all parsers (#551)f27f2ceperf: refactorparameterCountto optimize performance (#591)ccad155dep: upgrade [email protected] (#588)f75bd25chore: update test dependencies (#585)0f12509fix(docs): replace var with let or const in ReadMe (#581)5e6dd08fix(docs): remove security file (#590)d127b9cci: uselcovonlyreporter for thetest-ciscript (#584)d0bf2befix: remove skip of test (#589)You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill 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 versionwill 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 dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.