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

Hammock dependency #1

Open
vicb opened this issue Jul 17, 2014 · 1 comment
Open

Hammock dependency #1

vicb opened this issue Jul 17, 2014 · 1 comment

Comments

@vicb
Copy link

vicb commented Jul 17, 2014

I haven't look at the details but is the dep on hammock really required, couldn't this be a generic mapper ?

@vsavkin
Copy link
Owner

vsavkin commented Jul 17, 2014

You are right. The library consists of a generic mapper and a hammock configuration builder, and I was thinking about splitting them. I, however, decided to wait a bit, because I'm planning to add such annotations as @ResourceId and @errors:

class Post {
  @ResourceId() String secretId;
  @Errors() List someErrors;
}

These annotations are hammock-specific. I think it is possible to keep hammock-specific annotations and generic ones separate. If it is the case, I'm going to extract the mapper into its own library.

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

No branches or pull requests

2 participants