Skip to content

Conversation

@bparrishMines
Copy link
Contributor

@bparrishMines bparrishMines commented May 20, 2020

A recent pub publish check requires a non release Dart SDK, bumping the requirement so we don't get the warning.

This change is a no-op as the minimally required Flutter version for v2 embedding is (1.12.13+hotfix.5) and it already requires Dart 2.7.0.

@bparrishMines bparrishMines requested a review from amirh as a code owner May 20, 2020 00:45
@bparrishMines bparrishMines changed the title Bum dart version requirement [firebase_admob] Bump dart version requirement May 20, 2020
@bparrishMines bparrishMines requested review from blasten and cyanglaz May 20, 2020 00:46
Copy link

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

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

LGTM!

@bparrishMines bparrishMines merged commit 9befb6a into firebase:master May 20, 2020
@bparrishMines bparrishMines deleted the dart_version_bump branch May 20, 2020 23:25
@firebase firebase locked and limited conversation to collaborators Jul 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants