Skip to content

docs: correct Fabric dependency provider requirement to React Native 0.77 - #383

Merged
thomson-t merged 1 commit into
mainfrom
fix/sample-rn-077-comment
Sep 4, 2026
Merged

docs: correct Fabric dependency provider requirement to React Native 0.77#383
thomson-t merged 1 commit into
mainfrom
fix/sample-rn-077-comment

Conversation

@jamesnrokt

Copy link
Copy Markdown
Contributor

Summary

The Fabric dependency provider is required from React Native 0.77, not 0.76.
RCTDependencyProvider, the ReactAppDependencyProvider pod, and
RCTAppDelegate.dependencyProvider are all absent at v0.76.0 and added at
v0.77.0, where thirdPartyFabricComponents starts consulting the provider.

Corrects the version in the sample AppDelegate comment and the matching iOS
setup callout in the README, both of which contradicted MIGRATING.md. The
>= 0.76.0 peer dependency floor is a separate claim and is unchanged.

Testing Plan

Comment and prose only, no code paths touched. Prettier and markdownlint pass
with no new issues.

…0.77

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@jamesnrokt
jamesnrokt requested a review from a team as a code owner September 4, 2026 14:45
Copilot AI lite review requested due to automatic review settings September 4, 2026 14:45
@cursor

cursor Bot commented Sep 4, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation and sample comments only; no runtime or build behavior changes.

Overview
Updates iOS setup guidance so the Fabric dependency provider (RCTAppDependencyProvider / self.dependencyProvider) is documented as required from React Native 0.77+, not 0.76+.

The README callout and the comment in sample/ios/MParticleSample/AppDelegate.mm now match MIGRATING.md, which notes those APIs and thirdPartyFabricComponents integration arrive in 0.77. The peer dependency floor (>= 0.76.0) is unchanged.

Reviewed by Cursor Bugbot for commit 944f28c. Bugbot is set up for automated code reviews on this repo. Configure here.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are comment/prose-only and make the documentation consistent with the existing migration guidance.

Pull request overview

Updates the project documentation to correctly state that the iOS Fabric dependency provider requirement begins in React Native 0.77 (not 0.76), aligning the sample code comment and README guidance with MIGRATING.md.

Changes:

  • Update the sample iOS AppDelegate comment to say “Required since React Native 0.77”.
  • Update the README iOS setup callout to say “React Native 0.77+ requires a Fabric dependency provider”.
File summaries
File Description
sample/ios/MParticleSample/AppDelegate.mm Corrects the inline comment about when dependencyProvider becomes required.
README.md Corrects the iOS setup callout to match the 0.77 requirement described in migration guidance.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@thomson-t
thomson-t merged commit 9149d93 into main Sep 4, 2026
12 checks passed
@thomson-t
thomson-t deleted the fix/sample-rn-077-comment branch September 4, 2026 15:41
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.

3 participants