Skip to content

[analytics] migrate to core Dart Plugin reporting#8730

Merged
pq merged 1 commit into
flutter:mainfrom
pq:analytics_migrateToDP
Jan 27, 2026
Merged

[analytics] migrate to core Dart Plugin reporting#8730
pq merged 1 commit into
flutter:mainfrom
pq:analytics_migrateToDP

Conversation

@pq
Copy link
Copy Markdown
Collaborator

@pq pq commented Jan 26, 2026

Dart 502 introduces the core analytics implementation and this migrates our stub implementations to use it. We remove the now defunct UnifiedAnalytics implementation to keep things tidy.

Note that this does introduce more version coupling. Once published we'll have more places that will break in the presence of a pre-502 Dart plugin. (Since this was already happening with the PubRoot cleanup, e.g., #8725, I didn't consider doing version testing and any defensive programming; if folks feel strongly we can revisit though!)

Follow-up from #8598


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

@pq pq requested a review from helin24 January 26, 2026 23:59
@pq pq merged commit 6f73f07 into flutter:main Jan 27, 2026
8 checks passed
@pq pq deleted the analytics_migrateToDP branch January 27, 2026 17:06
@helin24
Copy link
Copy Markdown
Member

helin24 commented Jan 27, 2026

Great! I agree we probably don't need to worry about the old Dart version issue.

@pq
Copy link
Copy Markdown
Collaborator Author

pq commented Jan 27, 2026

Cool, thanks! I'm a little surprised that IDEA doesn't support explicit version dependencies but since they don't I think occasional issues with version skew are probably tolerable (if not expected). Worth keeping an eye out though!

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