Skip to content

Conversation

@mCodex
Copy link
Owner

@mCodex mCodex commented Aug 12, 2025

Upgraded React Native and related dependencies to 0.81.x in both root and example projects. Updated Android build tools, compile/target SDK versions, and Kotlin version. Refactored MainApplication to use new React Native entry point. Enabled edge-to-edge display support in gradle.properties. Updated Gradle wrapper to 8.14.3 and fixed gradlew scripts for compatibility.

Upgraded React Native and related dependencies to 0.81.x in both root and example projects. Updated Android build tools, compile/target SDK versions, and Kotlin version. Refactored MainApplication to use new React Native entry point. Enabled edge-to-edge display support in gradle.properties. Updated Gradle wrapper to 8.14.3 and fixed gradlew scripts for compatibility.
Copilot AI review requested due to automatic review settings August 12, 2025 16:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades React Native from version 0.80.x to 0.81.x and updates related dependencies across both the root project and example project. The upgrade includes modernizing Android build configurations and adopting new React Native initialization patterns.

Key changes:

  • Upgraded React Native from 0.80.x to 0.81.x with related dependency updates
  • Updated Android build tools, SDK versions, and Kotlin version for compatibility
  • Refactored MainApplication to use the new React Native entry point pattern

Reviewed Changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Updated React Native and related dev dependencies to 0.81.x versions
lefthook.yml Commented out commit-msg hooks (likely temporarily for development)
example/package.json Upgraded React Native and dependencies in example project
example/android/gradlew.bat Updated Gradle wrapper script for Windows with new execution pattern
example/android/gradlew Updated Unix Gradle wrapper script with new execution pattern
example/android/gradle/wrapper/gradle-wrapper.properties Upgraded Gradle wrapper to version 8.14.3
example/android/gradle.properties Added edge-to-edge display support configuration
example/android/build.gradle Updated Android build tools, SDK versions, and Kotlin version
example/android/app/src/main/java/plugpagnitro/example/MainApplication.kt Refactored to use new React Native initialization pattern

esac

CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar
CLASSPATH="\\\"\\\""
Copy link

Copilot AI Aug 12, 2025

Choose a reason for hiding this comment

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

The CLASSPATH is set to an empty quoted string which will likely cause Gradle execution to fail. This should either be removed entirely or set to a proper value.

Suggested change
CLASSPATH="\\\"\\\""

Copilot uses AI. Check for mistakes.

@rem Execute Gradle
"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*
"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %*
Copy link

Copilot AI Aug 12, 2025

Choose a reason for hiding this comment

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

Setting CLASSPATH to empty may cause issues with Gradle execution. This change should be verified to ensure it works correctly with the new jar-based execution pattern.

Copilot uses AI. Check for mistakes.
@mCodex mCodex merged commit 986b3d3 into main Aug 14, 2025
2 of 4 checks passed
@mCodex mCodex deleted the feat/rn081 branch November 4, 2025 14:26
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