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
Make sure methods annotated with @Funq
are declared as public
#27670
Make sure methods annotated with @Funq
are declared as public
#27670
Conversation
looks good to me |
This comment has been minimized.
This comment has been minimized.
c2d06b8
to
582e1ea
Compare
one nit: missing test |
sure, I'll add the test. |
582e1ea
to
41a61a8
Compare
Here you go. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense
41a61a8
to
b992535
Compare
Rebased onto current main as other PR changed pom.xml. |
This comment has been minimized.
This comment has been minimized.
In light of the test failures, are we sure that |
Only test that's failing is the one I added on Friday here #27674. It tests exactly what it should, but I keep forgetting to declare methods public. It's my fault, but validation is good thing. I'll fix in a minute. |
b992535
to
51cffe3
Compare
fixes: #27669
Throw an exception and break the build when non-public methods are annotated with
@Funq
as only public methods are registered as functions. This will give developer the information he made mistake during the build as there is no good reason to declare methods with@Funq
and don't use them as Funq functions.