Skip to content

Commit 475b137

Browse files
committed
Mark interruptionLevel as @nonnull
1 parent d1e30fc commit 475b137

File tree

5 files changed

+10
-9
lines changed

5 files changed

+10
-9
lines changed

firebase-appdistribution-api/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistribution.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,8 @@ public interface FirebaseAppDistribution {
206206
* below Android 8, this corresponds to a notification channel importance and once set cannot
207207
* be changed except by the user.
208208
*/
209-
void showFeedbackNotification(int infoTextResourceId, InterruptionLevel interruptionLevel);
209+
void showFeedbackNotification(
210+
int infoTextResourceId, @NonNull InterruptionLevel interruptionLevel);
210211

211212
/**
212213
* Displays a notification that, when tapped, will take a screenshot of the current activity, then
@@ -234,7 +235,7 @@ public interface FirebaseAppDistribution {
234235
* be changed except by the user.
235236
*/
236237
void showFeedbackNotification(
237-
@NonNull CharSequence infoText, InterruptionLevel interruptionLevel);
238+
@NonNull CharSequence infoText, @NonNull InterruptionLevel interruptionLevel);
238239

239240
/**
240241
* Hides the notification shown with {@link #showFeedbackNotification}.

firebase-appdistribution-api/src/main/java/com/google/firebase/appdistribution/internal/FirebaseAppDistributionProxy.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,13 +97,13 @@ public void startFeedback(@NonNull CharSequence infoText, @Nullable Uri screensh
9797

9898
@Override
9999
public void showFeedbackNotification(
100-
int infoTextResourceId, InterruptionLevel interruptionLevel) {
100+
int infoTextResourceId, @NonNull InterruptionLevel interruptionLevel) {
101101
delegate.showFeedbackNotification(infoTextResourceId, interruptionLevel);
102102
}
103103

104104
@Override
105105
public void showFeedbackNotification(
106-
@NonNull CharSequence infoText, InterruptionLevel interruptionLevel) {
106+
@NonNull CharSequence infoText, @NonNull InterruptionLevel interruptionLevel) {
107107
delegate.showFeedbackNotification(infoText, interruptionLevel);
108108
}
109109

firebase-appdistribution-api/src/main/java/com/google/firebase/appdistribution/internal/FirebaseAppDistributionStub.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,11 +87,11 @@ public void startFeedback(@NonNull CharSequence infoText, @Nullable Uri screensh
8787

8888
@Override
8989
public void showFeedbackNotification(
90-
int infoTextResourceId, InterruptionLevel interruptionLevel) {}
90+
int infoTextResourceId, @NonNull InterruptionLevel interruptionLevel) {}
9191

9292
@Override
9393
public void showFeedbackNotification(
94-
@NonNull CharSequence infoText, InterruptionLevel interruptionLevel) {}
94+
@NonNull CharSequence infoText, @NonNull InterruptionLevel interruptionLevel) {}
9595

9696
@Override
9797
public void cancelFeedbackNotification() {}

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/impl/FirebaseAppDistributionImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -354,13 +354,13 @@ public void startFeedback(@NonNull CharSequence infoText, @Nullable Uri screensh
354354

355355
@Override
356356
public void showFeedbackNotification(
357-
int infoTextResourceId, InterruptionLevel interruptionLevel) {
357+
int infoTextResourceId, @NonNull InterruptionLevel interruptionLevel) {
358358
showFeedbackNotification(getText(infoTextResourceId), interruptionLevel);
359359
}
360360

361361
@Override
362362
public void showFeedbackNotification(
363-
@NonNull CharSequence infoText, InterruptionLevel interruptionLevel) {
363+
@NonNull CharSequence infoText, @NonNull InterruptionLevel interruptionLevel) {
364364
notificationsManager.showFeedbackNotification(infoText, interruptionLevel);
365365
}
366366

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/impl/FirebaseAppDistributionNotificationsManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ private static int getPendingIntentFlags(int baseFlags) {
130130
}
131131

132132
public void showFeedbackNotification(
133-
@NonNull CharSequence infoText, InterruptionLevel interruptionLevel) {
133+
@NonNull CharSequence infoText, @NonNull InterruptionLevel interruptionLevel) {
134134
// Create the NotificationChannel, but only on API 26+ because
135135
// the NotificationChannel class is new and not in the support library
136136
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {

0 commit comments

Comments
 (0)