Skip to content

Remove unused com.facebook.react.uimanager.MeasureSpecAssertions import#3846

Merged
mfazekas merged 1 commit into
rnmapbox:mainfrom
mateoguzmana:main
May 26, 2025
Merged

Remove unused com.facebook.react.uimanager.MeasureSpecAssertions import#3846
mfazekas merged 1 commit into
rnmapbox:mainfrom
mateoguzmana:main

Conversation

@mateoguzmana

@mateoguzmana mateoguzmana commented Apr 28, 2025

Copy link
Copy Markdown
Contributor

Description

I'm currently looking into reducing visibility to the public API surface of React Native. I've noticed you are one of the few repos using com.facebook.react.uimanager.MeasureSpecAssertions; this import is currently unused in this codebase hence I'd like to remove it to reduce breaking changes across the ecosystem.

Checklist

  • I've read CONTRIBUTING.md
  • In V11 mode/android
  • In New Architecture mode/android

Screenshot OR Video

N/A

Component to reproduce the issue you're fixing

N/A

@mateoguzmana mateoguzmana marked this pull request as ready for review April 28, 2025 17:19
@mateoguzmana mateoguzmana changed the title Remove unused com.facebook.react.uimanager.MeasureSpecAssertions Remove unused com.facebook.react.uimanager.MeasureSpecAssertions import Apr 28, 2025
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mateoguzmana mateoguzmana temporarily deployed to CI with Mapbox Tokens May 26, 2025 05:17 — with GitHub Actions Inactive
@mfazekas mfazekas merged commit 8d12536 into rnmapbox:main May 26, 2025
7 of 14 checks passed
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.

2 participants