Skip to content

Conversation

greg-at-moderne
Copy link
Contributor

What's changed?

Adding an extra argument to AddDependencyVisitor constructor calls.

What's your motivation?

Fix broken CI after
openrewrite/rewrite#5371

/home/runner/work/rewrite-migrate-java/rewrite-migrate-java/src/main/java/org/openrewrite/java/migrate/javax/AddJaxbRuntime.java:163: error: constructor AddDependencyVisitor in class AddDependencyVisitor cannot be applied to given types;
                        g = (G.CompilationUnit) new org.openrewrite.gradle.AddDependencyVisitor(groupId, artifactId, version, null, "runtimeOnly", null, null, null, null)
                                                ^

@greg-at-moderne greg-at-moderne self-assigned this Jun 11, 2025
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Jun 11, 2025
@greg-at-moderne greg-at-moderne requested a review from sambsnyd June 11, 2025 07:32
@greg-at-moderne greg-at-moderne merged commit fa60311 into main Jun 11, 2025
2 checks passed
@greg-at-moderne greg-at-moderne deleted the greg-fix-AddDependencyVisitor branch June 11, 2025 07:43
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant