Skip to content

Fix ReflectionMethodUse: Remove reflection in AnimationDriver error message#55359

Closed
mdvacca wants to merge 1 commit intofacebook:mainfrom
mdvacca:export-D91736698
Closed

Fix ReflectionMethodUse: Remove reflection in AnimationDriver error message#55359
mdvacca wants to merge 1 commit intofacebook:mainfrom
mdvacca:export-D91736698

Conversation

@mdvacca
Copy link
Contributor

@mdvacca mdvacca commented Jan 29, 2026

Summary:
Fixed ReflectionMethodUse lint error in AnimationDriver.kt.

Replaced javaClass.simpleName with a hardcoded string to avoid reflection
which can cause issues with Redex obfuscation in production builds.

changelog: [internal] internal

Differential Revision: D91736698

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 29, 2026
@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Jan 29, 2026
Copy link
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@meta-codesync
Copy link

meta-codesync bot commented Jan 29, 2026

@mdvacca has exported this pull request. If you are a Meta employee, you can view the originating Diff in D91736698.

@mdvacca mdvacca force-pushed the export-D91736698 branch 2 times, most recently from 98697b7 to 04d701d Compare January 30, 2026 18:06
mdvacca added a commit to mdvacca/react-native that referenced this pull request Feb 3, 2026
…essage (facebook#55359)

Summary:

Fixed ReflectionMethodUse lint error in AnimationDriver.kt.

Replaced `javaClass.simpleName` with a hardcoded string to avoid reflection
which can cause issues with Redex obfuscation in production builds.

changelog: [internal] internal

Reviewed By: javache

Differential Revision: D91736698
mdvacca added a commit to mdvacca/react-native that referenced this pull request Feb 3, 2026
…essage (facebook#55359)

Summary:

Fixed ReflectionMethodUse lint error in AnimationDriver.kt.

Replaced `javaClass.simpleName` with a hardcoded string to avoid reflection
which can cause issues with Redex obfuscation in production builds.

changelog: [internal] internal

Reviewed By: javache

Differential Revision: D91736698
…essage (facebook#55359)

Summary:

Fixed ReflectionMethodUse lint error in AnimationDriver.kt.

Replaced `javaClass.simpleName` with a hardcoded string to avoid reflection
which can cause issues with Redex obfuscation in production builds.

changelog: [internal] internal

Reviewed By: javache

Differential Revision: D91736698
mdvacca added a commit to mdvacca/react-native that referenced this pull request Feb 3, 2026
…essage (facebook#55359)

Summary:

Fixed ReflectionMethodUse lint error in AnimationDriver.kt.

Replaced `javaClass.simpleName` with a hardcoded string to avoid reflection
which can cause issues with Redex obfuscation in production builds.

changelog: [internal] internal

Reviewed By: javache

Differential Revision: D91736698
@meta-codesync meta-codesync bot closed this in 485bbda Feb 4, 2026
@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @mdvacca in 485bbda

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Feb 4, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 4, 2026

This pull request has been merged in 485bbda.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants