-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[interactive_media_ads] Creates and adds the interactive_media_ads plugin
#6060
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
Merged
Merged
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
78ae502
initial flutter create
bparrishMines 6847016
add repo required stuff
bparrishMines b49ae74
fix pubspecs and add licenses
bparrishMines c1067a6
remove analysis options
bparrishMines 3f7f6bb
fix analyze warnings
bparrishMines b29653d
add podfile for example
bparrishMines c0cff54
add base platform interface classes
bparrishMines 0801f4c
fix analyze warning
bparrishMines c980a80
more impls
bparrishMines d2bde70
use verify token
bparrishMines e54e56a
add app facing interface
bparrishMines e4bdf45
implementation of addisplaycontainer
bparrishMines bab7c98
connect app facing to platform interface
bparrishMines 645753a
stuff
bparrishMines 3c60012
add documentation
bparrishMines 972c0ed
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 45e3fba
cleanup some dart
bparrishMines 16b0a7d
add first interface test
bparrishMines 6c1b1ad
test platform ad display container
bparrishMines e9f92db
ads manager delegate tests
bparrishMines 9c8ee61
add app facing tests
bparrishMines 8ae248a
finish tests
bparrishMines b6e2014
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 89d10a9
fix lint errors
bparrishMines 7a82249
formatting
bparrishMines 764d97b
fix more lints
bparrishMines c38bf50
update readme about development
bparrishMines 04eadac
update podspec
bparrishMines 4fa492e
try fix podspec again
bparrishMines 37c3fe3
formatting
bparrishMines aef9c3d
fix more lints
bparrishMines 1aa0455
try getting integration tests running
bparrishMines ef693a8
try gradle 8
bparrishMines 7574d7c
maybe fix
bparrishMines 52083c0
try fixing integration test runner
bparrishMines 45f57b0
remove espresso
bparrishMines 321ba99
change package name
bparrishMines c4ceda3
change dartintegrationteset location
bparrishMines c80733d
bump gradle for legacy project
bparrishMines ca7eb6a
for kotlin
bparrishMines d2e6974
add the dot
bparrishMines bdf655f
fix gradle
bparrishMines 0ebe298
try fix gradle
bparrishMines 6d135cd
lower versions for compilation
bparrishMines 8d31250
fix import
bparrishMines 7384eff
fix package name
bparrishMines 78b4067
add wake lock permission
bparrishMines 8d44136
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 4691fb0
add file
bparrishMines 0870a5a
try remaking plugin
bparrishMines 8e0c675
replace old with new
bparrishMines e5575bd
fix package name
bparrishMines e782cc3
formatting
bparrishMines 39dcf50
try to add integration test
bparrishMines 7ea9beb
add licenses
bparrishMines c90e582
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 78fe3b6
add test implementation runner
bparrishMines e191fd5
change file location
bparrishMines c790f46
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines a8785ef
fix integration test
bparrishMines 5e6c9e7
fix integration tests again
bparrishMines 350c216
try close system dialog
bparrishMines 68c854e
add to int test too i guess
bparrishMines ec017ba
try doing nothing in int test
bparrishMines 1fbd152
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 1e8a3c3
add launch android test
bparrishMines 4222ea2
use with text and not key
bparrishMines 9d8a55a
add a print
bparrishMines 93bc180
add print
bparrishMines 886bf26
add pragmas
bparrishMines ac4714c
fwoaie
bparrishMines 43e2023
undo
bparrishMines 331b163
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 6fc8d04
add kotlin to androidtest maybe
bparrishMines 9de555f
undo
bparrishMines 7fc6b5a
remove int test
bparrishMines ff73a65
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines f764bfb
update dart and flutter versions
bparrishMines 50e2f20
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 93e62fc
PR comments
bparrishMines 378b539
change platform to base
bparrishMines 3579326
update platform classes with base
bparrishMines 8c45df5
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 630ecba
remove todos
bparrishMines dfbb53d
make methods required
bparrishMines 25fc282
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines 476a4bc
since methods are required no tests are needed
bparrishMines 1fef696
Merge branch 'main' of github.com:flutter/packages into ima
bparrishMines File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| # Below is a list of people and organizations that have contributed | ||
| # to the Flutter project. Names should be added to the list like so: | ||
| # | ||
| # Name/Organization <email address> | ||
|
|
||
| Google Inc. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| ## 0.0.1 | ||
|
|
||
| * Adds platform interface for Android and iOS. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| Copyright 2013 The Flutter Authors. All rights reserved. | ||
|
|
||
| Redistribution and use in source and binary forms, with or without modification, | ||
| are permitted provided that the following conditions are met: | ||
|
|
||
| * Redistributions of source code must retain the above copyright | ||
| notice, this list of conditions and the following disclaimer. | ||
| * Redistributions in binary form must reproduce the above | ||
| copyright notice, this list of conditions and the following | ||
| disclaimer in the documentation and/or other materials provided | ||
| with the distribution. | ||
| * Neither the name of Google Inc. nor the names of its | ||
| contributors may be used to endorse or promote products derived | ||
| from this software without specific prior written permission. | ||
|
|
||
| THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND | ||
| ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | ||
| WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
| DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR | ||
| ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES | ||
| (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; | ||
| LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON | ||
| ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | ||
| (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | ||
| SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| # interactive\_media\_ads | ||
|
|
||
| Flutter plugin for the [Interactive Media Ads SDKs][1]. | ||
|
|
||
| [](https://pub.dev/packages/interactive_media_ads) | ||
|
|
||
| A Flutter plugin for using the Interactive Media Ads SDKs on Android and iOS. | ||
|
|
||
| | | Android | iOS | | ||
| |-------------|---------|-------| | ||
| | **Support** | SDK 19+ | 12.0+ | | ||
|
|
||
| **This package is still in development.** | ||
|
|
||
| [1]: https://developers.google.com/interactive-media-ads |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,78 @@ | ||
| group 'dev.flutter.packages.interactive_media_ads' | ||
| version '1.0-SNAPSHOT' | ||
|
|
||
| buildscript { | ||
| ext.kotlin_version = '1.7.10' | ||
| repositories { | ||
| google() | ||
| mavenCentral() | ||
| } | ||
|
|
||
| dependencies { | ||
| classpath 'com.android.tools.build:gradle:8.0.0' | ||
| classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version" | ||
| } | ||
| } | ||
|
|
||
| allprojects { | ||
| repositories { | ||
| google() | ||
| mavenCentral() | ||
| } | ||
| } | ||
|
|
||
| apply plugin: 'com.android.library' | ||
| apply plugin: 'kotlin-android' | ||
|
|
||
| android { | ||
| if (project.android.hasProperty("namespace")) { | ||
| namespace 'dev.flutter.packages.interactive_media_ads' | ||
| } | ||
|
|
||
| compileSdk 34 | ||
|
|
||
| compileOptions { | ||
| sourceCompatibility JavaVersion.VERSION_1_8 | ||
| targetCompatibility JavaVersion.VERSION_1_8 | ||
| } | ||
|
|
||
| kotlinOptions { | ||
| jvmTarget = '1.8' | ||
| } | ||
|
|
||
| sourceSets { | ||
| main.java.srcDirs += 'src/main/kotlin' | ||
| test.java.srcDirs += 'src/test/kotlin' | ||
| } | ||
|
|
||
| defaultConfig { | ||
| minSdk 19 | ||
| } | ||
|
|
||
| dependencies { | ||
| implementation 'androidx.annotation:annotation:1.5.0' | ||
| testImplementation 'junit:junit:4.13.2' | ||
| testImplementation 'org.jetbrains.kotlin:kotlin-test' | ||
| testImplementation 'org.mockito:mockito-inline:5.1.0' | ||
| testImplementation 'androidx.test:core:1.3.0' | ||
| } | ||
|
|
||
| lintOptions { | ||
| checkAllWarnings true | ||
| warningsAsErrors true | ||
| disable 'AndroidGradlePluginVersion', 'InvalidPackage', 'GradleDependency' | ||
| } | ||
|
|
||
| testOptions { | ||
| unitTests.includeAndroidResources = true | ||
| unitTests.returnDefaultValues = true | ||
| unitTests.all { | ||
| useJUnitPlatform() | ||
| testLogging { | ||
| events "passed", "skipped", "failed", "standardOut", "standardError" | ||
| outputs.upToDateWhen {false} | ||
| showStandardStreams = true | ||
| } | ||
| } | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| rootProject.name = 'interactive_media_ads' |
3 changes: 3 additions & 0 deletions
3
packages/interactive_media_ads/android/src/main/AndroidManifest.xml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| <manifest xmlns:android="http://schemas.android.com/apk/res/android" | ||
| package="dev.flutter.packages.interactive_media_ads"> | ||
| </manifest> |
37 changes: 37 additions & 0 deletions
37
...d/src/main/kotlin/dev/flutter/packages/interactive_media_ads/InteractiveMediaAdsPlugin.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| // Copyright 2013 The Flutter Authors. All rights reserved. | ||
| // Use of this source code is governed by a BSD-style license that can be | ||
| // found in the LICENSE file. | ||
|
|
||
| package dev.flutter.packages.interactive_media_ads | ||
|
|
||
| import io.flutter.embedding.engine.plugins.FlutterPlugin | ||
| import io.flutter.plugin.common.MethodCall | ||
| import io.flutter.plugin.common.MethodChannel | ||
| import io.flutter.plugin.common.MethodChannel.MethodCallHandler | ||
| import io.flutter.plugin.common.MethodChannel.Result | ||
|
|
||
| /** InteractiveMediaAdsPlugin */ | ||
| class InteractiveMediaAdsPlugin : FlutterPlugin, MethodCallHandler { | ||
| /// The MethodChannel that will the communication between Flutter and native Android | ||
| /// | ||
| /// This local reference serves to register the plugin with the Flutter Engine and unregister it | ||
| /// when the Flutter Engine is detached from the Activity | ||
| private lateinit var channel: MethodChannel | ||
|
|
||
| override fun onAttachedToEngine(flutterPluginBinding: FlutterPlugin.FlutterPluginBinding) { | ||
| channel = MethodChannel(flutterPluginBinding.binaryMessenger, "interactive_media_ads") | ||
| channel.setMethodCallHandler(this) | ||
| } | ||
|
|
||
| override fun onMethodCall(call: MethodCall, result: Result) { | ||
| if (call.method == "getPlatformVersion") { | ||
| result.success("Android ${android.os.Build.VERSION.RELEASE}") | ||
| } else { | ||
| result.notImplemented() | ||
| } | ||
| } | ||
|
|
||
| override fun onDetachedFromEngine(binding: FlutterPlugin.FlutterPluginBinding) { | ||
| channel.setMethodCallHandler(null) | ||
| } | ||
| } |
31 changes: 31 additions & 0 deletions
31
...c/test/kotlin/dev/flutter/packages/interactive_media_ads/InteractiveMediaAdsPluginTest.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| // Copyright 2013 The Flutter Authors. All rights reserved. | ||
| // Use of this source code is governed by a BSD-style license that can be | ||
| // found in the LICENSE file. | ||
|
|
||
| package dev.flutter.packages.interactive_media_ads | ||
|
|
||
| import io.flutter.plugin.common.MethodCall | ||
| import io.flutter.plugin.common.MethodChannel | ||
| import kotlin.test.Test | ||
| import org.mockito.Mockito | ||
|
|
||
| /* | ||
| * This demonstrates a simple unit test of the Kotlin portion of this plugin's implementation. | ||
| * | ||
| * Once you have built the plugin's example app, you can run these tests from the command | ||
| * line by running `./gradlew testDebugUnitTest` in the `example/android/` directory, or | ||
| * you can run them directly from IDEs that support JUnit such as Android Studio. | ||
| */ | ||
|
|
||
| internal class InteractiveMediaAdsPluginTest { | ||
| @Test | ||
| fun onMethodCall_getPlatformVersion_returnsExpectedValue() { | ||
| val plugin = InteractiveMediaAdsPlugin() | ||
|
|
||
| val call = MethodCall("getPlatformVersion", null) | ||
| val mockResult: MethodChannel.Result = Mockito.mock(MethodChannel.Result::class.java) | ||
| plugin.onMethodCall(call, mockResult) | ||
|
|
||
| Mockito.verify(mockResult).success("Android " + android.os.Build.VERSION.RELEASE) | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| # Platform Implementation Test App | ||
|
|
||
| This is a test app for manual testing and automated integration testing | ||
| of this platform implementation. It is not intended to demonstrate actual use of | ||
| this package, since the intent is that plugin clients use the app-facing | ||
| package. | ||
|
|
||
| Unless you are making changes to this implementation package, this example is | ||
| very unlikely to be relevant. |
68 changes: 68 additions & 0 deletions
68
packages/interactive_media_ads/example/android/app/build.gradle
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,68 @@ | ||
| plugins { | ||
| id "com.android.application" | ||
| id "kotlin-android" | ||
| id "dev.flutter.flutter-gradle-plugin" | ||
| } | ||
|
|
||
| def localProperties = new Properties() | ||
| def localPropertiesFile = rootProject.file('local.properties') | ||
| if (localPropertiesFile.exists()) { | ||
| localPropertiesFile.withReader('UTF-8') { reader -> | ||
| localProperties.load(reader) | ||
| } | ||
| } | ||
|
|
||
| def flutterVersionCode = localProperties.getProperty('flutter.versionCode') | ||
| if (flutterVersionCode == null) { | ||
| flutterVersionCode = '1' | ||
| } | ||
|
|
||
| def flutterVersionName = localProperties.getProperty('flutter.versionName') | ||
| if (flutterVersionName == null) { | ||
| flutterVersionName = '1.0' | ||
| } | ||
|
|
||
| android { | ||
| namespace "dev.flutter.packages.interactive_media_ads_example" | ||
| compileSdk flutter.compileSdkVersion | ||
| ndkVersion flutter.ndkVersion | ||
|
|
||
| compileOptions { | ||
| sourceCompatibility JavaVersion.VERSION_1_8 | ||
| targetCompatibility JavaVersion.VERSION_1_8 | ||
| } | ||
|
|
||
| kotlinOptions { | ||
| jvmTarget = '1.8' | ||
| } | ||
|
|
||
| sourceSets { | ||
| main.java.srcDirs += 'src/main/kotlin' | ||
| } | ||
|
|
||
| defaultConfig { | ||
| applicationId "dev.flutter.packages.interactive_media_ads_example" | ||
| minSdk flutter.minSdkVersion | ||
| targetSdk flutter.targetSdkVersion | ||
| versionCode flutterVersionCode.toInteger() | ||
| versionName flutterVersionName | ||
| testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner" | ||
| } | ||
|
|
||
| buildTypes { | ||
| release { | ||
| signingConfig signingConfigs.debug | ||
| } | ||
| } | ||
| } | ||
|
|
||
| flutter { | ||
| source '../..' | ||
| } | ||
|
|
||
| dependencies { | ||
| testImplementation 'junit:junit:4.13.2' | ||
| androidTestImplementation 'androidx.test:runner:1.2.0' | ||
| androidTestImplementation 'androidx.test.espresso:espresso-core:3.2.0' | ||
| api 'androidx.test:core:1.4.0' | ||
| } |
17 changes: 17 additions & 0 deletions
17
...androidTest/kotlin/dev/flutter/packages/interactive_media_ads_example/MainActivityTest.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| // Copyright 2013 The Flutter Authors. All rights reserved. | ||
| // Use of this source code is governed by a BSD-style license that can be | ||
| // found in the LICENSE file. | ||
|
|
||
| package dev.flutter.packages.interactive_media_ads_example | ||
|
|
||
| import androidx.test.rule.ActivityTestRule | ||
| import dev.flutter.plugins.integration_test.FlutterTestRunner | ||
| import io.flutter.plugins.DartIntegrationTest | ||
| import org.junit.Rule | ||
| import org.junit.runner.RunWith | ||
|
|
||
| @DartIntegrationTest | ||
| @RunWith(FlutterTestRunner::class) | ||
| class MainActivityTest { | ||
| @JvmField @Rule var rule = ActivityTestRule(MainActivity::class.java) | ||
| } |
20 changes: 20 additions & 0 deletions
20
packages/interactive_media_ads/example/android/app/src/debug/AndroidManifest.xml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| <manifest xmlns:android="http://schemas.android.com/apk/res/android"> | ||
| <!-- The INTERNET permission is required for development. Specifically, | ||
| the Flutter tool needs it to communicate with the running application | ||
| to allow setting breakpoints, to provide hot reload, etc. | ||
| --> | ||
| <uses-permission android:name="android.permission.INTERNET"/> | ||
| <application android:usesCleartextTraffic="true"> | ||
| <activity | ||
| android:name="dev.flutter.packages.interactive_media_ads_example.DriverExtensionActivity" | ||
| android:launchMode="singleTop" | ||
| android:theme="@style/LaunchTheme" | ||
| android:configChanges="orientation|keyboardHidden|keyboard|screenSize|smallestScreenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode" | ||
| android:hardwareAccelerated="true" | ||
| android:windowSoftInputMode="adjustResize"> | ||
| <meta-data | ||
| android:name="io.flutter.Entrypoint" | ||
| android:value="integrationTestMain" /> | ||
| </activity> | ||
| </application> | ||
| </manifest> |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.