-
Notifications
You must be signed in to change notification settings - Fork 50
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
@ConfigProperty for java.util.Duration with defaultValue flagged as error #1207
Comments
MicroProfileConfigASTValidator is Microprofile specific, but Quarkus projects use a specific DurationConverter. MicroProfileConfigASTValidator would need to load converters in a dynamic way (using special classloader) then we could validate via reflection, but that's not a trivial solution. Or we take the dirty road and simply copy DurationConverter over, use it directly in MicroProfileConfigASTValidator (since it's only used for Quarkus projects anyways) and fix this issue. Duration conversion would still need to be implemented at the QuarkusConfigMappingProvider / QuarkusConfigPropertiesProvider / QuarkusConfigRootProvider level so duration properties can show up and be validated in application.properties |
error Fixes redhat-developer#1207 Signed-off-by: azerr <[email protected]>
error Fixes redhat-developer#1207 Signed-off-by: azerr <[email protected]>
error Fixes redhat-developer#1207 Signed-off-by: azerr <[email protected]>
error Fixes redhat-developer#1207 Signed-off-by: azerr <[email protected]>
same issue as redhat-developer/vscode-quarkus#631:
The text was updated successfully, but these errors were encountered: