-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[google_maps_flutter_web]: Update instructions to reflect current versions #4163
Conversation
test-exempt: only affects documentation |
google_maps_flutter: ^0.5.28 | ||
google_maps_flutter_web: ^0.1.0 | ||
google_maps_flutter: ^2.0.0 | ||
google_maps_flutter_web: ^0.3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should just remove this code section, and adjust the line above to say something like: [...] so you'll need to [add it explicitly](https://pub.dev/packages/google_maps_flutter_web/install)
We should get out of the business of having doomed-to-bitrot installation examples in our READMEs, and let pub.dev's dynamic page handle that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Very good change, I've just merged it. Thanks!! |
The
google_maps_flutter_web
packageREADME.md
lists old versions in the usage description. This is confusing to users attempting to use Google Maps in Flutter for web apps.fixes: flutter/flutter#86631
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist
dart format
.)[shared_preferences]
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.