Skip to content

convert Spring Boot application-*.properties to application-*.yaml#1066

Open
barbulescu wants to merge 10 commits into
openrewrite:mainfrom
barbulescu:properties-2-yaml
Open

convert Spring Boot application-*.properties to application-*.yaml#1066
barbulescu wants to merge 10 commits into
openrewrite:mainfrom
barbulescu:properties-2-yaml

Conversation

@barbulescu

@barbulescu barbulescu commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

What's changed?

Created a recipe for migrating pplication-.properties to application-.yaml.

What's your motivation?

Anything in particular you'd like reviewers to focus on?

Did I miss any test cases? Can it be done simpler?

Anyone you would like to review specifically?

@timtebeek

Checklist

  • I've added unit tests to cover both positive and negative cases
  • I've read and applied the recipe conventions and best practices
  • I've used the IntelliJ IDEA auto-formatter on affected files

@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Jul 10, 2026
@timtebeek
timtebeek self-requested a review July 13, 2026 15:24
@barbulescu
barbulescu marked this pull request as draft July 13, 2026 19:26
@barbulescu

Copy link
Copy Markdown
Contributor Author

@timtebeek I tried to cleanup a bit the initial version and I did a rebase to fix merge conflicts - please update

@barbulescu

Copy link
Copy Markdown
Contributor Author

This test started failing randomly but it does look to be related to my changes:

ReplaceExtendWithAndContextConfigurationTest > extendWithContextConfigurationKeptWhenUsingLoaderArgument() FAILED
    java.lang.IllegalStateException: LST contains missing or invalid type information
    Identifier->FieldAccess->Assignment->Annotation->ClassDeclaration->CompilationUnit
    /*~~(Identifier type is missing or malformed)~~>*/SpringBootContextLoader
    https://docs.openrewrite.org/reference/faq#im-seeing-lst-contains-missing-or-invalid-type-information-in-my-recipe-unit-tests-how-to-resolve
        at org.openrewrite.java.Assertions.assertValidTypes(Assertions.java:131)
        at org.openrewrite.java.Assertions.validateTypes(Assertions.java:68)
        at org.openrewrite.java.Assertions$$Lambda/0x000000900139d548.accept(Unknown Source)
        at org.openrewrite.test.RewriteTest.rewriteRun(RewriteTest.java:350)
        at org.openrewrite.test.RewriteTest.rewriteRun(RewriteTest.java:159)
        at org.openrewrite.test.RewriteTest.rewriteRun(RewriteTest.java:154)
        at org.openrewrite.java.spring.boot2.ReplaceExtendWithAndContextConfigurationTest.extendWithContextConfigurationKeptWhenUsingLoaderArgument(ReplaceExtendWithAndContextConfigurationTest.java:80)

@barbulescu
barbulescu marked this pull request as ready for review July 14, 2026 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant