Skip to content
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

Fix check for configuration properties #11379

Conversation

altro3
Copy link
Contributor

@altro3 altro3 commented Nov 24, 2024

Fixes #10943

@altro3
Copy link
Contributor Author

altro3 commented Nov 24, 2024

@dstepanov look to this, pls. I've added tests for inject-java and inject-kotlin. As you see, test with kotlin (KSP) is failed. Wrong introdpection @ConfigurationProperties bean. injectibleFields list is empty, bust must contains levelOnez property. With inject java all works fine

@dstepanov dstepanov self-assigned this Nov 25, 2024
@altro3
Copy link
Contributor Author

altro3 commented Nov 28, 2024

@dstepanov It doesn't look like it's a checkstyle error :-)

{036D2E05-CDDF-4EF4-AA24-06A9D48A72E4}

@altro3
Copy link
Contributor Author

altro3 commented Nov 28, 2024

@dstepanov looks like the 4.8.x branch is broken

@dstepanov
Copy link
Contributor

Looks like my fix is not correct

@dstepanov dstepanov changed the title Test for #10943 Fix check for configuration properties Nov 29, 2024
@graemerocher graemerocher added the type: bug Something isn't working label Nov 29, 2024
@graemerocher graemerocher merged commit 0b6624a into micronaut-projects:4.8.x Nov 29, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants