Bump node-fetch and react-native in /example#27
Draft
dependabot[bot] wants to merge 1 commit intomasterfrom
Draft
Bump node-fetch and react-native in /example#27dependabot[bot] wants to merge 1 commit intomasterfrom
dependabot[bot] wants to merge 1 commit intomasterfrom
Conversation
Removes [node-fetch](https://github.com/node-fetch/node-fetch). It's no longer used after updating ancestor dependency [react-native](https://github.com/facebook/react-native/tree/HEAD/packages/react-native). These dependencies need to be updated together. Removes `node-fetch` Updates `react-native` from 0.58.4 to 0.84.0 - [Release notes](https://github.com/facebook/react-native/releases) - [Changelog](https://github.com/facebook/react-native/blob/main/CHANGELOG-0.5x.md) - [Commits](https://github.com/facebook/react-native/commits/v0.84.0/packages/react-native) --- updated-dependencies: - dependency-name: node-fetch dependency-version: dependency-type: indirect - dependency-name: react-native dependency-version: 0.84.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| "@twotalltotems/react-native-otp-input": "^1.3.7", | ||
| "react": "16.6.3", | ||
| "react-native": "0.58.4" | ||
| "react-native": "0.84.0" |
There was a problem hiding this comment.
PR missing linked Linear issue per team rules
Low Severity
This PR has no linked Linear issue (or similar issue tracking reference) in the body, as required by team rules. Since this is an automated Dependabot security fix PR, a brief justification comment explaining that no Linear ticket is needed for automated dependency updates would satisfy the requirement.
Triggered by team rule: Linear Alignment
|
No reviewers assigned, converting to draft |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Removes node-fetch. It's no longer used after updating ancestor dependency react-native. These dependencies need to be updated together.
Removes
node-fetchUpdates
react-nativefrom 0.58.4 to 0.84.0Release notes
Sourced from react-native's releases.
... (truncated)
Changelog
Sourced from react-native's changelog.
... (truncated)
Commits
5693f60Release 0.84.05c69e21Release 0.84.0-rc.53552d65[LOCAL] Bump Hermes versionc5bb0b1Release 0.84.0-rc.4a24f1d8fix(hermes-inspector-modern): Apply HERMES_V1_ENABLED define to all build typ...8d561b0Include final (redirected to) URL in response object. Fixes #55248. (#55251)8206c12fix(react-native): add prefabPublishing to hermes-engine for build-from-sourc...af7f3deRelease 0.84.0-rc.30dd7aadAddtransformOriginprop support to theAnimationBackend. (#55203)aeed939Add support forbackfaceVisibilityand addflextoAnimatedPropsBuilder...Maintainer changes
This version was pushed to npm by react-native-bot, a new releaser for react-native since your current version.
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 rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@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)You can disable automated security fix PRs for this repo from the Security Alerts page.
Note
Medium Risk
Large React Native major-version bump in the
exampleapp can introduce build/runtime regressions across iOS/Android and Metro tooling; change is dependency-scoped but impacts core platform behavior.Overview
Linked issue(s): None referenced.
What changed: Updates the
exampleapp’s dependencies by upgradingreact-nativefrom0.58.4to0.84.0and removingnode-fetch(no longer used).Acceptance criteria:⚠️ Not stated in PR.
Scope validation: ✅ Change appears limited to dependency updates/removal in
example.Test coverage:⚠️ No new/updated tests; relies on existing
jestand manual build/run verification.Blockers/concerns: Ensure the
exampleapp still builds and runs on iOS/Android after the RN upgrade (native config/toolchain changes may be required).Out-of-scope suggestion: Add CI smoke builds for
example(Android + iOS) to catch future RN/tooling breakages early.Written by Cursor Bugbot for commit d4a7619. This will update automatically on new commits. Configure here.