Skip to content

chore(iOS): remove guards regarding RN < 0.82 version#4062

Merged
sgaczol merged 2 commits into
mainfrom
@sgaczol/rn-version-minor-guard
May 21, 2026
Merged

chore(iOS): remove guards regarding RN < 0.82 version#4062
sgaczol merged 2 commits into
mainfrom
@sgaczol/rn-version-minor-guard

Conversation

@sgaczol

@sgaczol sgaczol commented May 19, 2026

Copy link
Copy Markdown
Collaborator

Description

According to peerDependencies we only support RN versions >= 0.82.0.
This PR removes guards regarding lower versions.

Changes

  • remove REACT_NATIVE_VERSION_MINOR >= 82 guards
  • remove REACT_NATIVE_VERSION_MINOR < 82 guards and code executed on this condition
  • remove RNS_REACT_NATIVE_VERSION_MINOR_BELOW_78 flag in RNSDefines.h

Before & after - visual documentation

N/A

Test plan

Build example apps.

Checklist

  • Included code example that can be used to test this change.
  • For visual changes, included screenshots / GIFs / recordings documenting the change.
  • For API changes, updated relevant public types.
  • Ensured that CI passes

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kligarski kligarski 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.

Looks good. Make sure that the app builds in debug/release and please check tests related to bottom accessory.

Comment thread ios/utils/RNSDefines.h Outdated
@sgaczol sgaczol merged commit a8394a4 into main May 21, 2026
6 checks passed
@sgaczol sgaczol deleted the @sgaczol/rn-version-minor-guard branch May 21, 2026 08:30
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.

4 participants