-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
Allow generation of external Meta-Bean #178
Comments
Our internal guides say that autogenerated code should not be committed to version control. So I always find it strange when all the autogenerated JodaBeans are committed in full. When I find some time I'll take a look at the maven plugin and see if it's possible to change it to generate the code under target instead of adding to the original file. Similar to how other code generator tools work |
I think it's quite clear that we can implement this sort of thing in the respective generators but I was more trying to understand first whether there are technical reasons for the status quo :-) |
Joda-Beans was intended for a use case where code checked out of a source repo should compile immediately (without any need to run a tool like an annotation processor or maven). The code isn't that well structured to break out a meta-model for pluggable generators, however that would very much be a good direction to go. As for the amount of code, the light and minimal bean styles generate less code and may be of interest. Particularly as of the latest release. |
Thanks for jumping in! |
|
I wonder if it would be a good idea to optionally allow the generation of external Meta-Beans (just the Meta-Bean in its own class)?
This would allow for not having to check in generated code into version control, keep the bean classes more compact and closer to a plain POJO (except the annotations of course and yes, of course they are also sort of plain with all the generated code...)
On the flipside one would always have to run the generation step upon a fresh checkout, but I don't think this is much of a drawback (some people might also argue it's the cleaner way).
The text was updated successfully, but these errors were encountered: