Skip to content

Allow to specify MoxyReflector package and use it from another module#117

Merged
senneco merged 2 commits intoArello-Mobile:masterfrom
ekursakov:multi-module-moxy-reflector
May 4, 2017
Merged

Allow to specify MoxyReflector package and use it from another module#117
senneco merged 2 commits intoArello-Mobile:masterfrom
ekursakov:multi-module-moxy-reflector

Conversation

@ekursakov
Copy link
Copy Markdown
Contributor

@ekursakov ekursakov commented May 3, 2017

Добавил возможность выбора пакета в котором будет сгенерирован MoxyReflector (#72).
Имя пакета задается через аргументы annotation processor'а (как здесь):

kapt {
    arguments {
        arg("moxyReflectorPackage", "com.example.common")
    }
}

Если оно не указано, то используется com.arellomobile.mvp и в нем генерируется MoxyReflector, который также включает в себя данные из других MoxyReflector'ов, зарегистрированных с помощью аннотации @RegisterMoxyReflectorPackages (идея отсюда):

@RegisterMoxyReflectorPackages({"com.example.common"})
public class App extends Application {
...

@senneco senneco merged commit 5167d89 into Arello-Mobile:master May 4, 2017
@senneco
Copy link
Copy Markdown
Collaborator

senneco commented May 4, 2017

Эх, только надо было из ветки develop фичу стартовать. Жаль я поздно заметил.

@ekursakov
Copy link
Copy Markdown
Contributor Author

У меня ветка из develop была, а вот в пулреквесте по умолчанию master стоит, а я как-то это пропустил.

@ekursakov
Copy link
Copy Markdown
Contributor Author

Есть еще некоторые сомнения по поводу конфигурации, сейчас получилось что часть параметров задается через аргументы процессора, а модули регистрируются через аннотацию.
Может стоит сделать все параметрами процессора, или как-то еще по другому?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants