Skip to content
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

Proguard #188

Open
vladislav-h opened this issue Feb 16, 2023 · 3 comments
Open

Proguard #188

vladislav-h opened this issue Feb 16, 2023 · 3 comments
Labels
transmitted Issue transmitted to inner service

Comments

@vladislav-h
Copy link

Мне кажется, что proguard конфиг библиотеки можно улучшить. Там написано:
-keep class com.yandex.mobile.ads.** { *; }
Наверно это слишком общее и из-за этого остается много "мертвого" кода.
Такое же предложение было и раньше, там лучше описано:
#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
Copy link
Collaborator

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

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

@vladislav-h
Copy link
Author

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

@mobile-ads-github
Copy link
Collaborator

Спасибо за Ваше обращение, для разбора создана задача с номером 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.

@mobile-ads-github mobile-ads-github added the transmitted Issue transmitted to inner service label Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
transmitted Issue transmitted to inner service
Projects
None yet
Development

No branches or pull requests

2 participants