-
Notifications
You must be signed in to change notification settings - Fork 4k
Add pedantic to all Firebase plugins #2040
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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! Please make sure the version updates are correct before merging.
Thanks a lot for doing this!
@@ -1,3 +1,7 @@ | |||
## 0.13.4 |
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.
should it be 0.13.3+1?
## 0.13.4 | |
## 0.13.3+1 |
@@ -3,7 +3,7 @@ description: | |||
Flutter plugin for Cloud Firestore, a cloud-hosted, noSQL database with | |||
live synchronization and offline support on Android and iOS. | |||
homepage: https://github.com/FirebaseExtended/flutterfire/tree/master/packages/cloud_firestore/cloud_firestore | |||
version: 0.13.3 | |||
version: 0.13.4 |
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.
version: 0.13.4 | |
version: 0.13.3+1 |
@@ -1,3 +1,7 @@ | |||
## 0.4.3 |
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.
## 0.4.3 | |
## 0.4.2+1 |
@@ -1,6 +1,6 @@ | |||
name: cloud_functions | |||
description: Flutter plugin for Cloud Functions. | |||
version: 0.4.2 | |||
version: 0.4.3 |
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.
version: 0.4.3 | |
version: 0.4.2+1 |
@@ -1,3 +1,7 @@ | |||
## 0.15.6 |
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.
## 0.15.6 | |
## 0.15.5+1 |
@@ -3,7 +3,7 @@ description: Flutter plugin for Firebase Auth, enabling Android and iOS | |||
authentication using passwords, phone numbers and identity providers | |||
like Google, Facebook and Twitter. | |||
homepage: https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_auth/firebase_auth | |||
version: 0.15.5 | |||
version: 0.15.6 |
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.
version: 0.15.6 | |
version: 0.15.5+1 |
@@ -1,3 +1,7 @@ | |||
## 0.4.5 |
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.
## 0.4.5 | |
## 0.4.4+1 |
@@ -2,7 +2,7 @@ name: firebase_core | |||
description: Flutter plugin for Firebase Core, enabling connecting to multiple | |||
Firebase apps. | |||
homepage: https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_core/firebase_core | |||
version: 0.4.4 | |||
version: 0.4.5 |
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.
version: 0.4.5 | |
version: 0.4.4+1 |
@@ -1,3 +1,7 @@ | |||
## 0.1.4 |
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.
## 0.1.4 | |
## 0.1.3+1 |
@@ -2,7 +2,7 @@ name: firebase_crashlytics | |||
description: | |||
Flutter plugin for Firebase Crashlytics. It reports uncaught errors to the | |||
Firebase console. | |||
version: 0.1.3 | |||
version: 0.1.4 |
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.
version: 0.1.4 | |
version: 0.1.3+1 |
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 once @cyanglaz's comments are addressed and tests pass. Thank you SO much.
ff3ca32
to
c6823da
Compare
Description
Adds pedantic dependencies for all Firebase plugins'
analysis_options
.Related Issues
Copy of flutter/plugins#2543
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?