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

Annotation support #67

Open
wants to merge 5 commits into
from

Conversation

Projects
None yet
7 participants
@yigit

yigit commented Jan 2, 2013

adds support for custom annotations on generated classes, constructors, fields and getter/setter methods. issue #66.

this pull request is a re-implementation from Path's fork of GreenDao . specifically commit d3deaf

I could not make the eclipse formatting work in Intellij 12 so sorry about the formatting diff.

@onyxmueller

This comment has been minimized.

Show comment
Hide comment
@onyxmueller

onyxmueller Jun 5, 2013

I think adding annotation support to greenDAO is a fantastic idea. Is there any reason this pull request or functionality in general has not been implemented? (This is more for the greenDAO maintainers.)

I think adding annotation support to greenDAO is a fantastic idea. Is there any reason this pull request or functionality in general has not been implemented? (This is more for the greenDAO maintainers.)

@green-coder

This comment has been minimized.

Show comment
Hide comment
@green-coder

green-coder Jul 6, 2013

Apparently, the author of the project doesn't seem to respond to any pull request. If we want the project to evolve, maybe we have to maintain it by ourself and merge features from different branches by ourself too.

Apparently, the author of the project doesn't seem to respond to any pull request. If we want the project to evolve, maybe we have to maintain it by ourself and merge features from different branches by ourself too.

@greenrobot

This comment has been minimized.

Show comment
Hide comment
@greenrobot

greenrobot Jul 10, 2013

Owner

Thanks for your contribution. Please check here for a contributor agreement: http://greendao-orm.com/2013/07/09/community-contributions/

Owner

greenrobot commented Jul 10, 2013

Thanks for your contribution. Please check here for a contributor agreement: http://greendao-orm.com/2013/07/09/community-contributions/

@SebastianEngel

This comment has been minimized.

Show comment
Hide comment
@SebastianEngel

SebastianEngel Feb 7, 2014

Is there any progress on that? Annotation support is a great and useful addition.

Is there any progress on that? Annotation support is a great and useful addition.

@green-coder

This comment has been minimized.

Show comment
Hide comment
@green-coder

green-coder Feb 8, 2014

I signed the contributor agreement, sent it to @greenrobot, and after that I didn't hear any news from him.

I signed the contributor agreement, sent it to @greenrobot, and after that I didn't hear any news from him.

@castylx

This comment has been minimized.

Show comment
Hide comment
@castylx

castylx Feb 12, 2014

I need this badly, has anyone used this fork in production code?

castylx commented Feb 12, 2014

I need this badly, has anyone used this fork in production code?

@yigit

This comment has been minimized.

Show comment
Hide comment
@yigit

yigit Feb 12, 2014

I don't want to drive anybody away from the main project, but if you need, you can use our fork @path
https://github.com/path/greenDAO
we've been using it on production for more than a year

yigit commented Feb 12, 2014

I don't want to drive anybody away from the main project, but if you need, you can use our fork @path
https://github.com/path/greenDAO
we've been using it on production for more than a year

@castylx

This comment has been minimized.

Show comment
Hide comment
@castylx

castylx Feb 12, 2014

Cool, thanks!

castylx commented Feb 12, 2014

Cool, thanks!

carloshwa pushed a commit to doubleencore/greenDAO that referenced this pull request Jul 10, 2014

@carloshwa carloshwa referenced this pull request Jul 10, 2014

Merged

Annotation support #2

@tobiasschuerg

This comment has been minimized.

Show comment
Hide comment
@tobiasschuerg

tobiasschuerg Aug 25, 2014

Please merge this into the main project!

Please merge this into the main project!

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