Skip to content

Conversation

@pq
Copy link
Collaborator

@pq pq commented Dec 11, 2025

Migrates API from the flutter plugin (and refines it).

Still TODO, the actual communication (See UnifiedAnalyticsReporter.process(..)).

Once implemented, instrumentation of an action (for example, DartPubActionBase), will look like this:

  override fun actionPerformed(e: AnActionEvent) {
    val (module, pubspecYamlFile) = getModuleAndPubspecYamlFile(e) ?: return

    Analytics.report(AnalyticsData.forAction(this, e))
    performPubAction(module, pubspecYamlFile, true)
  }

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

@pq pq requested review from helin24 and jwren December 11, 2025 18:23
@pq pq merged commit 8ee1769 into flutter:main Dec 11, 2025
9 checks passed
@pq pq deleted the analytics_reportingAPI branch December 11, 2025 21:06
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