Update react-router-dom 6.27.0 → 6.28.0 (minor) #254
Merged
Conversation
This file contains 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
![depfu[bot]](https://avatars.githubusercontent.com/in/715?s=60&v=4){kind=small}
|
By default, I don't review pull requests opened by bots. If you would like me to review this pull request anyway, you can request a review via the |
|
Review changes with Changed Files
|
Reviewer's Guide by SourceryThis PR updates the react-router-dom dependency from version 6.27.0 to 6.28.0, along with its indirect dependency react-router. The update also introduces three new CommonJS-specific dependencies: string-width-cjs, strip-ansi-cjs, and wrap-ansi-cjs. Class diagram for updated dependencies in package.jsonclassDiagram
class PackageJson {
react-dom : ^18.3.1
bootstrap : ^5.3.3
jquery : 3.7.1
react-router-dom : ^6.28.0
react-router-bootstrap : ^0.26.3
react-scripts : ^5.0.1
reactstrap : ^9.2.3
string-width-cjs : 4.2.3
strip-ansi-cjs : 6.0.1
wrap-ansi-cjs : 7.0.0
}
note for PackageJson "Updated react-router-dom to 6.28.0 and added new dependencies"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
👋 Hi there!Everything looks good!
|
Feedback
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Please double check the following review of the pull request:Issues counts
Changes in the diff
Identified Issues
No issues were identified in the incoming changes. Missing TestsSince the changes primarily involve updating package versions, no specific tests are necessary for the code itself. However, it is recommended to run the existing test suite to ensure that the updated packages do not introduce any regressions or compatibility issues. If the project does not have a test suite, consider implementing tests for critical functionalities to ensure stability after dependency updates. Summon me to re-review when updated! Yours, Gooroo.dev |
github-actions
bot
added
the
size/L
Potential issues, bugs, and flaws that can introduce unwanted behavior.
Code suggestions and improvements for better exception handling, logic, standardization, and consistency.
|
|
Infisical secrets check: 🚨 Secrets leaked! Caution The Infisical CLI tool found secrets leaked in your repository. 💻 Scan logs🔎 Detected secrets in your GIT historyWarning The above table only displays the first 10 leaked secrets. 🐾 Secrets fingerprintTip If you want to ignore these leaked secrets, add the above fingerprint content to a file named |
|
Report too large to display inline |
|
Report is too large to display inline. Next stepsMark a package as acceptable riskTo ignore an alert, reply with a comment starting with
|
gstraccini
bot
added
the
☑️ auto-merge
![gstraccini[bot]](https://avatars.githubusercontent.com/in/480132?s=60&v=4){kind=small}
guibranco
approved these changes
Nov 8, 2024
|
@depfu merge |
|
guibranco
deleted the
depfu/update/POCUploadStreamClientApp/npm/react-router-dom-6.28.0
branch
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.
Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.
What changed?
✳️ react-router-dom (6.27.0 → 6.28.0) · Repo · Changelog
🆕 string-width-cjs (added, npm:[email protected])
🆕 strip-ansi-cjs (added, npm:[email protected])
🆕 wrap-ansi-cjs (added, npm:[email protected])
Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with
@depfu rebase.All Depfu comment commands
Summary by Sourcery
Build: