-
Notifications
You must be signed in to change notification settings - Fork 31
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
Annotations Processor #4
Comments
Hi Stokito, did anything happen with your JPatterns Annotations Processor? |
Hi @kabutz unfortunately no, I didn't finished a work. If you interested we can continue work on project and release it. |
It's nice to see people are still interested! |
You'll be really surprised how often I still sell my Design Patterns Course (I certainly am). We even did one last week in Paris. |
Hello, I created an annotation processor that checks some properties of annotated classes, methods and fields. If it thinks there is a bug in design pattern implementation, it reports a compilation warning (or error, or nothing - it can be configured). The compilation message can be produced for example if ConcreteDecorator class is abstract, or if it and it's superclasses don't contain any field of Component type. It is currently in my fork of JPatterns: https://github.com/aczaplinski/jpatterns . This fork also merges all important changes from all forks of JPatterns here, upgrades Java to version 11 and fixes some bugs. You can see the annotation processor in action in my Mockito fork: https://github.com/aczaplinski/mockito and my Guava fork: https://github.com/aczaplinski/guava . You have to provide the compiled version of my JPatterns fork for them to work. What do you think of it? |
Hi,
JFYI: I started a new project https://github.com/stokito/jpatterns-annotations-processor
My idea is to merge this project http://www.dp4j.com/ that already have processors of Singleton and template Method. But it has GNU license and I'll ask author to import the project to GitHub and republish it under Apache v2 licence.
If anybody wants to contribute - you welcome.
The text was updated successfully, but these errors were encountered: