flutter / flutter/flutter-intellij

Clean up plugin build process

Open
#8,348 0 comments 0 reactions 1 assignee Claimed by @helin24 View on GitHub
infra P1
Dominant language
Java
Stars
2k
Forks
356
Avg merge
1d 11h
Merged PRs (30d)
27

Description

While reading about plugin structure as part of decisions around dart plugin, we decided to reorganize the flutter plugin as well (mostly from @jwren). The goal is to make plugin building easier to understand and easier to debug by following conventions of IntelliJ plugin setup and gradle and removing complex paths that were added for historical reasons but are no longer needed.

- [x] remove build files in flutter-idea and flutter-studio subdirectories - instead of using a module structure for intellij vs Android studio plugins, we should be able to specify code to include/exclude in plugin.xml files https://github.com/flutter/flutter-intellij/pull/8278
- [x] flatten directories to remove flutter-idea and flutter-studio (https://github.com/flutter/flutter-intellij/pull/8343, https://github.com/flutter/flutter-intellij/pull/8384)
- [x] Remove required Android studio plugin dependency https://github.com/flutter/flutter-intellij/pull/7949
- [x] Consider how to support multiple platform versions - this could mean generating a single plugin that spans versions, using branches, or keeping the status quo (product-matrix.json). The complex build process for multiple versions was more necessary when platform APIs were changing quickly, but may not be needed now. (https://github.com/flutter/flutter-intellij/pull/8398)
- [x] Reduce dependency on plugin.dart and related files - we are doing a lot of customized file reading and writing and other tasks that could likely be replaced with gradle and IntelliJ platform configuration. (https://github.com/flutter/flutter-intellij/pull/8360, https://github.com/flutter/flutter-intellij/pull/8363, https://github.com/flutter/flutter-intellij/pull/8387, https://github.com/flutter/flutter-intellij/pull/8400, https://github.com/flutter/flutter-intellij/pull/8407, https://github.com/flutter/flutter-intellij/pull/8411)
- [x] Use more conventional directory structure (https://github.com/flutter/flutter-intellij/pull/8397)

Related cleanup - https://github.com/flutter/flutter-intellij/pull/8326, https://github.com/flutter/flutter-intellij/pull/8327, https://github.com/flutter/flutter-intellij/pull/8416, https://github.com/flutter/flutter-intellij/pull/8424

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.