yandexmobile / yandex-ads-sdk-android

Other
80 stars 22 forks source link

Proguard #188

Open vladislav-h opened 1 year ago

vladislav-h commented 1 year ago

Мне кажется, что proguard конфиг библиотеки можно улучшить. Там написано: -keep class com.yandex.mobile.ads.* { ; } Наверно это слишком общее и из-за этого остается много "мертвого" кода. Такое же предложение было и раньше, там лучше описано: https://github.com/yandexmobile/yandex-ads-sdk-android/issues/4

Убрал этот ряд из proguard конфига на пробу, вроде все работает. Количество методов понизилось где-то на 8000. Как можно запрашивать тестовую рекламу, чтобы лучше проверить, что proguard не убрал чего-то нужного? Я знаю только способ с ad unit id "R-M-DEMO-interstitial", но там всегда возвращается одна и та же реклама и не позволяет хорошо все проверить. В идеале хотелось бы протестировать с настоящей рекламой, но только в тестовом режиме.

В нашем случае дошли до proguard конфига из-за того, что новые версии com.yandex.android:mobileads добавили очень много методов и мы стали перекидывать 64k method counts limit. Не хочется включать multidex из-за "мертвого" кода. Так же, не хочется раздувать apk без необходимости.

Спасибо!

mobile-ads-github commented 1 year ago

Добрый день, спасибо за предложение! Мы рассмотрим его командой. В целом мы не можем на данный момент гарантировать, что не возникнут проблемы при удалении этого keep правила из consumer ProGuard правил SDK, полноценное тестирование включает в себя большое количество тест кейсов.

Для дополнительного тестирования можно воспользоваться блоком R-M-DEMO-interstitial-video. Командой продумаем предложение по внедрению тестового режима для продовых блоков. Спасибо!

vladislav-h commented 1 year ago

Большое спасибо за ответ и помощь!

mobile-ads-github commented 1 year ago

Спасибо за Ваше обращение, для разбора создана задача с номером PCODESUP-2238. Мы ознакомимся и вернемся с ответом в течение 2 дней, возможно запросим дополнительную информацию.

Thank you for your message, a task with the number PCODESUP-2238 has been created for analysis. We will get acquainted and return with the answer within 2 days, we may request additional information.