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

Implement enum constant in kotlin element factory #9771

Closed
wants to merge 12 commits into from

Conversation

altro3
Copy link
Contributor

@altro3 altro3 commented Aug 23, 2023

No description provided.

micronaut-build and others added 12 commits August 18, 2023 14:17
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…onaut-projects#9748)

* refactor: Pattern matching for instance of  and String.formatted

Use this link to re-run the recipe: https://app.moderne.io/recipes/builder/E3weKifbL

Co-authored-by: Moderne <[email protected]>

* Fix naming conflicts, checkstyle, and add generics to Maps

---------

Co-authored-by: Moderne <[email protected]>
…naut-projects#9763)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
# Conflicts:
#	core/src/main/java/io/micronaut/core/convert/DefaultMutableConversionService.java
#	gradle.properties
#	inject/src/main/java/io/micronaut/context/env/PropertySourcePropertyResolver.java
#	inject/src/main/java/io/micronaut/inject/annotation/EnvironmentConvertibleValuesMap.java
@altro3
Copy link
Contributor Author

altro3 commented Aug 23, 2023

@dstepanov Hi! Could you help with this change? Without it we can't use micronaut-openapi with KSP for enums. But I did something wrong and now I have this exception in test:

изображение

Need help!

@dstepanov
Copy link
Contributor

Can you please include only necessary changes? The other once please add in another PR

@altro3
Copy link
Contributor Author

altro3 commented Aug 23, 2023

@dstepanov i think, it's better, if merge all changes from branch 4.1.x to 4.2.x. and after it you will see only my changes.

I don't know another way to sync new branch in fork. I need to create branch with the same name - 4.2.x, but I need to start this branch from 4.1.x. And in original repo ypu still didn't merge branch 4.1.x. Jsut merge 4.1.x to 4.2.x

@altro3
Copy link
Contributor Author

altro3 commented Aug 24, 2023

Continue here: #9774

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants