Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates build tooling and JavaScript dependencies, while also removing multiple platform scaffolds and prototype tooling from the repository.
Changes:
- Bump Android build tooling in
forui/example(AGP, Kotlin, Gradle wrapper) and switch Kotlin JVM target configuration style. - Update several
docs/npm dependencies (Next.js, React, Tailwind, Wrangler, etc.). - Remove various prototype/docs_snippets assets and the demo app’s Android/Windows/Linux platform directories.
Reviewed changes
Copilot reviewed 44 out of 51 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| forui/example/android/settings.gradle | Bumps AGP + Kotlin plugin versions for the example Android build. |
| forui/example/android/gradle/wrapper/gradle-wrapper.properties | Updates Gradle wrapper distribution URL. |
| forui/example/android/gradle.properties | Adds Android/Gradle property flags for the example project. |
| forui/example/android/app/build.gradle | Removes kotlin-android plugin and changes Kotlin JVM target configuration. |
| docs/package.json | Updates docs site dependencies and tooling versions. |
| docs_snippets/prototype/serve.py | Removes prototype local server script. |
| docs_snippets/prototype/json_viewer.html | Removes prototype JSON viewer HTML. |
| docs_snippets/macos/Podfile | Removes macOS CocoaPods config for docs snippets. |
| docs_snippets/ios/Podfile | Removes iOS CocoaPods config for docs snippets. |
| demo/windows/runner/win32_window.h | Removes Windows runner source (platform scaffold deletion). |
| demo/windows/runner/win32_window.cpp | Removes Windows runner source (platform scaffold deletion). |
| demo/windows/runner/utils.h | Removes Windows runner utility header. |
| demo/windows/runner/utils.cpp | Removes Windows runner utility source. |
| demo/windows/runner/Runner.rc | Removes Windows runner resources. |
| demo/windows/runner/runner.exe.manifest | Removes Windows runner manifest. |
| demo/windows/runner/resource.h | Removes Windows runner resource header. |
| demo/windows/runner/main.cpp | Removes Windows runner entrypoint. |
| demo/windows/runner/flutter_window.h | Removes Windows Flutter window wrapper header. |
| demo/windows/runner/flutter_window.cpp | Removes Windows Flutter window wrapper source. |
| demo/windows/runner/CMakeLists.txt | Removes Windows runner CMake configuration. |
| demo/windows/flutter/CMakeLists.txt | Removes Windows Flutter-managed CMake configuration. |
| demo/windows/CMakeLists.txt | Removes Windows project-level CMake configuration. |
| demo/windows/.gitignore | Removes Windows-specific gitignore. |
| demo/linux/runner/my_application.h | Removes Linux runner source (platform scaffold deletion). |
| demo/linux/runner/my_application.cc | Removes Linux runner source (platform scaffold deletion). |
| demo/linux/runner/main.cc | Removes Linux runner entrypoint. |
| demo/linux/runner/CMakeLists.txt | Removes Linux runner CMake configuration. |
| demo/linux/flutter/CMakeLists.txt | Removes Linux Flutter-managed CMake configuration. |
| demo/linux/CMakeLists.txt | Removes Linux project-level CMake configuration. |
| demo/linux/.gitignore | Removes Linux-specific gitignore. |
| demo/android/settings.gradle.kts | Removes demo Android Gradle settings (platform scaffold deletion). |
| demo/android/gradle/wrapper/gradle-wrapper.properties | Removes demo Android Gradle wrapper config. |
| demo/android/gradle.properties | Removes demo Android Gradle properties. |
| demo/android/build.gradle.kts | Removes demo Android root build script. |
| demo/android/app/src/profile/AndroidManifest.xml | Removes demo Android profile manifest. |
| demo/android/app/src/main/res/values/styles.xml | Removes demo Android theme resources. |
| demo/android/app/src/main/res/values-night/styles.xml | Removes demo Android night theme resources. |
| demo/android/app/src/main/res/drawable/launch_background.xml | Removes demo Android launch background resource. |
| demo/android/app/src/main/res/drawable-v21/launch_background.xml | Removes demo Android v21 launch background resource. |
| demo/android/app/src/main/kotlin/com/example/demo/MainActivity.kt | Removes demo Android MainActivity. |
| demo/android/app/src/main/AndroidManifest.xml | Removes demo Android main manifest. |
| demo/android/app/src/debug/AndroidManifest.xml | Removes demo Android debug manifest. |
| demo/android/app/build.gradle.kts | Removes demo Android app Gradle build script. |
| demo/android/.gitignore | Removes demo Android-specific gitignore. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
1
to
4
| plugins { | ||
| id "com.android.application" | ||
| id "kotlin-android" | ||
| id "dev.flutter.flutter-gradle-plugin" | ||
| } |
Comment on lines
81
to
+83
| "eslint": "^9.39.4", | ||
| "eslint-config-next": "16.1.6", | ||
| "postcss": "^8.5.12", | ||
| "eslint-config-next": "16.2.4", | ||
| "postcss": "^8.5.14", |
Comment on lines
20
to
24
| plugins { | ||
| id "dev.flutter.flutter-plugin-loader" version "1.0.0" | ||
| id "com.android.application" version '8.13.2' apply false | ||
| id "org.jetbrains.kotlin.android" version "2.3.20" apply false | ||
| id "com.android.application" version '9.2.1' apply false | ||
| id "org.jetbrains.kotlin.android" version "2.3.21" apply false | ||
| } |
Contributor
Documentation Preview
You're seeing this because the docs/samples were updated. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe the changes
Checklist