Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Graduate admob plugin to beta #538

Merged
merged 6 commits into from
May 7, 2018
Merged

Graduate admob plugin to beta #538

merged 6 commits into from
May 7, 2018

Conversation

mit-mit
Copy link
Member

@mit-mit mit-mit commented May 5, 2018

No description provided.

@@ -4,8 +4,7 @@ A plugin for [Flutter](https://flutter.io) that supports loading and
displaying banner, interstitial (full-screen), and rewarded video ads using the
[Firebase AdMob API](https://firebase.google.com/docs/admob/).

*Warning*: This plugin is still under development, some AdMob features are not
available yet and testing has been limited.
*Note*: This plugin is is beta, and may still have a few issues and missing APIs.
Copy link
Contributor

Choose a reason for hiding this comment

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

Fix "is is"

Copy link
Contributor

@mravn-google mravn-google left a comment

Choose a reason for hiding this comment

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

LGTM

@collinjackson
Copy link
Contributor

Is there a reason why you choose this version number? 0.5.4+1 would be more accurate under semantic versioning since this is a documentation-only change.

@mit-mit
Copy link
Member Author

mit-mit commented May 7, 2018

I thought we used that (0.9) as a convention for packages that hit beta?

@collinjackson
Copy link
Contributor

I was not aware of such a convention. If one exists I'd love to know more so I can do the same for other beta plugins. Per our conversation I'll land it as 0.5.4+1 but feel free to bump to 0.9 in the future if it makes sense to do so.

@collinjackson collinjackson merged commit a090d17 into master May 7, 2018
slightfoot pushed a commit to slightfoot/plugins that referenced this pull request Jun 5, 2018
@mit-mit mit-mit deleted the mit-mit-admob branch July 13, 2018 11:21
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
4 participants