We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The @ConfigProperty has a target like
@Target({METHOD, FIELD, PARAMETER, TYPE})
Do we currently support METHOD and TYPE? Looks like in most cases, it's for fields.
For version, I would only keep FIELD and PARAMETER that we can easily support
Happy to push a PR if we are ok.
The text was updated successfully, but these errors were encountered:
I would say that CONSTRUCTOR and METHOD are more appropriate than TYPE as those two targets would potentially be used by those in a CDI environment.
Sorry, something went wrong.
Yes, I'm not sure myself why it's TYPE, but TYPE is really cool as it allows us to apply it to interfaces!
In Geronimo-config we used this to implement interface-style config a la Apache DeltaSpike. See #12
We should probably rename it to something else? @Configured for example?
@Configured
No branches or pull requests
The @ConfigProperty has a target like
Do we currently support METHOD and TYPE?
Looks like in most cases, it's for fields.
For version, I would only keep FIELD and PARAMETER that we can easily support
Happy to push a PR if we are ok.
The text was updated successfully, but these errors were encountered: