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

Add library config options #124

Merged
merged 3 commits into from
Feb 14, 2017
Merged

Add library config options #124

merged 3 commits into from
Feb 14, 2017

Conversation

elihart
Copy link
Contributor

@elihart elihart commented Feb 13, 2017

This adds support for optional library wide configuration options to the annotation processor. The two options available so far are requiring abstract model classes and requiring fields with @EpoxyAttribute to implement hashCode (or use hash=false).

This is done via a new @EpoxyConfig annotation that can be used anywhere in the project. My thinking is that an empty class could be created to do this. Something like

@EpoxyConfig(...)
final class EpoxyConfig {
  private EpoxyConfig(){}
}

I haven't seen any other ways for specifying options to an annotation processor, if someone has better ideas on how to do this let me know!

@felipecsl @seanabraham @gpeal @ngsilverman

@elihart elihart merged commit 3a3bc6a into master Feb 14, 2017
@elihart elihart deleted the eli/config branch February 14, 2017 19:58
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.

1 participant