Skip to content

SC-44615 Update react-intl to v10#664

Open
gregaubert wants to merge 1 commit intomainfrom
greg/update-peer-dependency-react-intl
Open

SC-44615 Update react-intl to v10#664
gregaubert wants to merge 1 commit intomainfrom
greg/update-peer-dependency-react-intl

Conversation

@gregaubert
Copy link
Copy Markdown
Member

Part of

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 31, 2026

Deploy Preview for echoes-react ready!

Name Link
🔨 Latest commit 72178f0
🔍 Latest deploy log https://app.netlify.com/projects/echoes-react/deploys/69cbdc87938b31000844addc
😎 Deploy Preview https://deploy-preview-664--echoes-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown

hashicorp-vault-sonar-prod bot commented Mar 31, 2026

SC-44615

@sonar-review-alpha
Copy link
Copy Markdown

Summary

Updates react-intl dependency from v6 to v10 (4 major versions). This is a significant version bump that may introduce breaking changes. The diff shows only package.json and yarn.lock modifications—no application code changes are visible in this PR, suggesting either the codebase is already compatible with v10's API or compatibility work was done previously.

What reviewers should know

Start by checking:

  • The repository uses react-intl extensively (FormattedMessage, useIntl hooks, etc. in ~30+ components)
  • Since no source code changes are shown, verify that v10 compatibility is already baked in or that migration was completed on another branch

Key review concerns:

  • React-intl v6→v10 spans 4 major versions with potential breaking changes; familiarize yourself with the migration guide if unfamiliar
  • Confirm this passes all CI checks (tests, linting, builds) to ensure the bump doesn't break the application
  • Check the yarn.lock changes match expected dependency tree adjustments

For context:

  • The commit message references SC-44615; check if there are related issues or acceptance criteria
  • The package.json also shows react@^19.0.0, so this is a modern React codebase already using current versions

  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@sonarqube-next
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Copy Markdown

@sonar-review-alpha sonar-review-alpha bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✅

Clean, minimal change — only the peer dependency version range and lock file entry are updated. No application code changes are needed, indicating the codebase is already compatible with react-intl v10.

🗣️ Give feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant