Skip to content

Remove boilerplate for restaurant finder example, as it is moving to a2ui, to be closer to the agent.#885

Merged
polina-c merged 3 commits intoflutter:mainfrom
polina-c:remove-app
Apr 29, 2026
Merged

Remove boilerplate for restaurant finder example, as it is moving to a2ui, to be closer to the agent.#885
polina-c merged 3 commits intoflutter:mainfrom
polina-c:remove-app

Conversation

@polina-c
Copy link
Copy Markdown
Collaborator

No description provided.

@polina-c polina-c changed the title Remove boilerplate for restaurant finder example, as it is moving to a2ui, to be closer to agent. Remove boilerplate for restaurant finder example, as it is moving to a2ui, to be closer to the agent. Apr 23, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request primarily involves the removal of the restaurant_finder project files across multiple platforms. It also includes manual updates to auto-generated configuration files in the simple_chat example, specifically adding the jni plugin to Linux and Windows build scripts and updating the macOS Podfile.lock. Feedback across all review comments correctly points out that these generated files should not be edited manually; instead, any necessary dependencies should be declared in the pubspec.yaml file to ensure consistency and prevent changes from being overwritten by the Flutter tool.

)

list(APPEND FLUTTER_FFI_PLUGIN_LIST
jni
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.

medium

This file is automatically generated by the Flutter tool and should not be modified manually, as indicated by the header on line 2. Manual changes will be lost when the tool is next run. If the jni plugin is a required dependency, it should be declared in the pubspec.yaml file instead of being added here.

)

list(APPEND FLUTTER_FFI_PLUGIN_LIST
jni
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.

medium

This file is automatically generated by the Flutter tool and should not be modified manually. Manual changes will be lost when the tool is next run. If the jni plugin is a required dependency, it should be declared in the pubspec.yaml file instead of being added here.

@@ -1,21 +1,35 @@
PODS:
- audioplayers_darwin (0.0.1):
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.

medium

The addition of audioplayers_darwin and video_player_avfoundation to this lock file without corresponding entries in examples/simple_chat/pubspec.yaml suggests that this generated file may have been modified manually or is out of sync with the project's dependency declarations. These dependencies should be managed through pubspec.yaml and updated via the standard pod install command.

@polina-c polina-c requested a review from jacobsimionato April 25, 2026 18:36
@polina-c polina-c merged commit 1a44cd0 into flutter:main Apr 29, 2026
30 checks passed
@polina-c polina-c deleted the remove-app branch April 29, 2026 22:44
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