Так что на будущее

окич

done

Не было ответа в MR, а обсуждение уже resolved, тут перезадам вопрос: я могу просто запушить поднятие версии API до 26 в отдельном MR или сначала таску на это создать лучше?

И, кстати, более общий вопрос: если какой-то мелкий баг на глаза попался (например, используется штука из 21 API без проверки, когда minApi 19), то создавать таску и фиксить или можно просто MR отправить на фикс?

Можно просто мр, если уверены

И ещё я так и не понял по поводу переменной окружения для номера билда, в MR откомментил (https://gitlab.com/terrakok/gitlab-client/merge_requests/25#note_41488026).

@dmitvitalii
про номер билда: там неслучайно все так сделано, зря вы начали править. скоро гитлаб выкатит нормальный номер, а пока надо оставить как есть. Ваш вариант будет давать разные номера сборки для каждого локального репозитория.
про ресурсы: зачем ваш презентер дает вьюхе строку по частям?? надо ее собрать в презентере и отдать вью уже готовую строку

Так мой вариант от вашего не отличается ничем. Отличие только в том, что он отрефакторен и переписан на груви. Он точно так же берёт из переменной окружения значение, проверяет на null, если он, то вернуть 1, иначе вернуть номер билда как строку.

Касательно строки: поправлю, тупанул.
Я ж сначала айди передавал, потом втупую на строку поменял.

ваш вариант будет запускаться на локальном репозитории, а мой нет. только на гитлабе. а локально всегда вернет 1

Мой вариант тоже локально всегда 1 вернёт, и ваш вариант локально вернёт то число, которое будет в переменной окружения, если его задать (завтра протестирую чтобы быть уверенным).
Я привел пример в скрине, если мы явно задаём эту переменную (т.е. тот кейс, когда билд-система таки засунула туда какое-то значение). Но я это сделал прям в терминале, из которого дёрнул ./gradlew installDebug потом.

Либо я все ещё не понимаю суть проблемы.

@terrakok Чnjбы заинжектить класс MarkDownConverter из сторонней библиотеки anddown в интерактор, нужно ли прописывать что-то дополнительно в gradle или Module toothpick? Не могу понять в чем трабла

toothpick.registries.NoFactoryFoundException: No factory could be found for class ru.terrakok.gitlabclient.model.interactor.project.MarkDownConverter. Check that the class has either a @Inject annotated constructor or contains @Inject annotated members. If using Registries, check that they are properly setup with annotation processor arguments.

надо в моддуле прописать биндинг

toothpick.registries.NoFactoryFoundException: No factory could be found for class ru.terrakok.gitlabclient.model.interactor.project.MarkDownConverter. Check that the class has either a @Inject annotated constructor or contains @Inject annotated members. If using Registries, check that they are properly setup with annotation processor arguments.

Это значит, что класс, который предоставляет библиотека, не конструктора с аннотацией @Inject =>.надо вручную прописать создание инстанса

именно 👍

либо через провайдер

если там надо что-то еще в конструктор передать

Вот этот класс надо заинжектить

class MarkDownConverter {
    private val converter = AndDown()

    fun markdownToHtml(raw: String) = converter.markdownToHtml(raw)
}

В модуле пишу

bind(MarkDownConverter::class.java)

Верно?

в модуле надо написать bind(MarkDownConverter::class.java).toInstance(MarkDownConverter())