search

yoomoney / yookassa-android-sdk

This library allows implementing payment acceptance into mobile apps on Android. It works as an extension to the YooMoney API.
https://yookassa.ru/developers/payment-forms/android-sdk
MIT License
40 stars 22 forks source link

Бесконечный цикл проверок code inspection при добавлении зависимости библиотеки #79

Open ArsMarks opened 3 years ago

ArsMarks commented 3 years ago

Прошелся по всем пунктам вашей инструкции по ссылке https://github.com/yoomoney/yookassa-android-sdk/blob/master/MIGRATION.md

После добавления этой зависимости implementation 'ru.yoomoney.sdk.kassa.payments:yookassa-android-sdk:6.0.2' Android Studio code analysis переходит в бесконечный цикл, если в классе присутствует аннотация @Inject constructor() в Kotlin файле. В Java файле подобное работает. Использую: Android Studio 4.2.1 Build #AI-202.7660.26.42.7351085, built on May 10, 2021 Runtime version: 11.0.8+0-b944-P17168821 amd64 VM: OpenJDK 64-Bit Server VM by N/A Linux 5.8.0-55-generic GC: G1 Young Generation, G1 Old Generation Memory: 2048M Cores: 4 Registry: external.system.auto.import.disabled=true Non-Bundled Plugins: org.jetbrains.kotlin Current Desktop: ubuntu:GNOME Также пробовал: Android Studio Bumblebee | 2021.1.1 Canary 2 Build #AI-211.6222.4.2111.7407564, built on May 28, 2021 Runtime version: 11.0.10+0-b96-7249189 amd64 VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o. Linux 5.8.0-55-generic GC: G1 Young Generation, G1 Old Generation Memory: 2048M Cores: 4 Registry: ide.instant.shutdown=false, external.system.auto.import.disabled=true_)

Current Desktop: ubuntu:GNOME

Gradle plugin 4.2.1 Kotlin plugin 203-1.5.10-release-891-AS7717.8

iskandarshabaev commented 3 years ago

Подскажите пожалуйста, а lint и Inspect code отрабатывают?

ArsMarks commented 3 years ago

Нет

iskandarshabaev commented 3 years ago

@ArsMarks Возможно у вас есть лог с ошибкой работы lint?

iskandarshabaev commented 3 years ago

Мне пока не удалось воспроизвести, завёл тестовый проектик с одной активити, подключил

ru.yoomoney.sdk.kassa.payments:yookassa-android-sdk:6.0.2
com.google.dagger:dagger:2.28.3
com.google.dagger:dagger-compiler:2.28.3

Завёл класс аннотацией @Inject к конструктору, lint отрабатывает с успехом. Есть возможность скинуть проект с примером где будет воспроизводится проблема?

ArsMarks commented 3 years ago

https://dropmefiles.com/SaHpb Вот пример проекта

iskandarshabaev commented 3 years ago

@ArsMarks Спасибо, смотрим

ArsMarks commented 3 years ago

Подтвердили такое поведение?

iskandarshabaev commented 3 years ago

@ArsMarks Пока что не удалось воспроизвести проблему.

iskandarshabaev commented 3 years ago

Единственное что вижу в правом верхнем углу экрана бесконечную надпись Analyzing...

image

Полагаю это то о чём вы говорите? Это блочит вам как-то релизы? Мешает разработке? Пытаюсь понять срочность задачи.

ArsMarks commented 3 years ago

Да, именно то, о чем я и говорил В больших классах/файлах при изменениях android lint работает очень долго (подсветка ошибок/ворнингов), либо не работает совсем Для нормальной работы, создаю отдельную ветку, где все зависимости и код вашей, использующий вашу библиотеку, закомментированы

iskandarshabaev commented 3 years ago

@ArsMarks Проблему нашли, починим в новом релизе.

ArsMarks commented 3 years ago

@iskandarshabaev Спасибо большое, будем ждать