Skip to content

Restructure Flutter plugin codebase into flutter/ directory for monorepo - #9101

Draft
helin24 wants to merge 2 commits into
flutter:mainfrom
helin24:monorepo-phase-1
Draft

Restructure Flutter plugin codebase into flutter/ directory for monorepo#9101
helin24 wants to merge 2 commits into
flutter:mainfrom
helin24:monorepo-phase-1

Conversation

@helin24

@helin24 helin24 commented Aug 28, 2026

Copy link
Copy Markdown
Member

Thanks for your contribution! Please replace this text with:

  • a description of what this PR is changing and why
  • any relevant issues
  • a description of how to verify the change is working
  • screenshots/gif if relevant

Review the contribution guidelines below:

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
  • I've included the required information in the description above.
  • My up-to-date information is in the AUTHORS file.
  • I've updated CHANGELOG.md if appropriate.
Contribution guidelines:
  • See
    our contributor guide and
    the Flutter organization contributor guide
    for general expectations for PRs.
  • Larger or significant changes should be discussed in an issue before creating a PR.
  • Dart contributions to our repos should follow the Dart style guide and use
    dart format.
  • Java and Kotlin contributions should strive to follow Java and Kotlin best
    practices (discussion).

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

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.

Code Review

This pull request restructures the repository into a monorepo by moving the Flutter plugin codebase, documentation, and agent skills into a new flutter/ subdirectory, updating configuration files like .gitignore and .github/dependabot.yml accordingly. The review feedback suggests correcting the root README.md to accurately describe the repository contents (as the Dart plugin is not present), updating an outdated branch reference in the CODE_OF_CONDUCT.md link, and removing redundant ignore patterns from the .gitignore file.

Comment thread README.md Outdated
Comment on lines +1 to +3
# Flutter and Dart IntelliJ Plugins

[![Latest plugin version](https://img.shields.io/jetbrains/plugin/v/9212)](https://plugins.jetbrains.com/plugin/9212-flutter)
[![Build Status](https://github.com/flutter/flutter-intellij/workflows/presubmit/badge.svg)](https://github.com/flutter/flutter-intellij/actions?query=branch%3Amain+workflow%3Apresubmit)
This repository is a monorepo containing the IntelliJ IDEA and Android Studio plugins for Flutter and Dart.

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

[CONCERN] The README states that this repository is a monorepo containing both the Flutter and Dart plugins. However, there is currently no Dart plugin codebase in this repository (only the flutter/ directory is present). If the Dart plugin is not part of this repository, we should update the description to avoid confusion.

Suggested change
# Flutter and Dart IntelliJ Plugins
[![Latest plugin version](https://img.shields.io/jetbrains/plugin/v/9212)](https://plugins.jetbrains.com/plugin/9212-flutter)
[![Build Status](https://github.com/flutter/flutter-intellij/workflows/presubmit/badge.svg)](https://github.com/flutter/flutter-intellij/actions?query=branch%3Amain+workflow%3Apresubmit)
This repository is a monorepo containing the IntelliJ IDEA and Android Studio plugins for Flutter and Dart.
# Flutter IntelliJ Plugin
This repository is a monorepo containing the IntelliJ IDEA and Android Studio plugins for Flutter.

Comment thread flutter/CODE_OF_CONDUCT.md Outdated
Comment on lines +3 to +4
When contributing code to the `flutter-intellij` project, please observe
Flutter's [code of conduct](https://github.com/flutter/flutter/blob/master/CODE_OF_CONDUCT.md).

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

[CONCERN] The link to Flutter's code of conduct uses the outdated master branch. Since the flutter/flutter repository has migrated to main as its default branch, we should update this link to use main to ensure long-term reliability.

Suggested change
When contributing code to the `flutter-intellij` project, please observe
Flutter's [code of conduct](https://github.com/flutter/flutter/blob/master/CODE_OF_CONDUCT.md).
When contributing code to the `flutter-intellij` project, please observe
Flutter's [code of conduct](https://github.com/flutter/flutter/blob/main/CODE_OF_CONDUCT.md).

Comment thread .gitignore Outdated
Comment on lines +47 to +49
.kotlin/
flutter-studio/build/
flutter-studio/.gradle/
flutter-idea/build/
flutter-idea/.gradle/
tool/plugin/.idea
**/.kotlin/
flutter/tool/plugin/.idea

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

[CONCERN] The entry flutter/tool/plugin/.idea is redundant because the global pattern **/.idea (defined at line 9) already ignores all .idea directories at any depth. Additionally, **/.kotlin/ at line 48 already covers .kotlin/ at line 47. We should remove these redundant entries to keep the .gitignore file clean and maintainable.

**/.kotlin/

@helin24
helin24 force-pushed the monorepo-phase-1 branch 2 times, most recently from 4d619cf to 943fd35 Compare August 29, 2026 04:25
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.

1 participant