Skip to content

Use fully reflection#99

Draft
cmoulliard wants to merge 15 commits intomainfrom
use-fully-reflection
Draft

Use fully reflection#99
cmoulliard wants to merge 15 commits intomainfrom
use-fully-reflection

Conversation

@cmoulliard
Copy link
Member

@cmoulliard cmoulliard commented Mar 9, 2026

  • Avoid to import the class: RewriteService, RewriteConfig and ClassLoaderUtils from the Quarkus - Picocli app
  • Use fully reflection

Result of the command rewrite executed on spring-boot-todo-app using rewrite-spring-to-quarkus: https://gist.github.com/cmoulliard/d296628381240100abaedf310940d312

@cmoulliard
WIP. Refactor the project to support to use fully the reflection
6108b14 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
WIP. Document the methods called using reflection
ec39e20 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Remove non needed comments around dependency
b25cbb2 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Simplify the readibility of the code of the method buildOpenRewriteEn…
8ab6954 
…vironment building the OpenRewrite Environment

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard cmoulliard marked this pull request as ready for review March 9, 2026 16:06
@cmoulliard
Change the scope of the utils dependency to inlude it part of the sha…
da5d8cb 
…ded service jar. Rename the package from toolbox to utils

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Added the link to the openrewrite issue for the test case: useRecipes…
c1a5883 
…JarToUpgradeDependencies

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Remove the non needed comments
93c59bd 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Remove the annotation of QuarkusTest as non needed within the test cl…
d0b2275 
…ass: RecipesJarTSVFileTest

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Moved the code of the test which should be launched using an isolated…
cd4147b 
… classloader within integration-tests and add RewriteServiceProxy to the utils module

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Remove the Disabled annotation
6f7838e 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard cmoulliard marked this pull request as draft March 10, 2026 06:26
@cmoulliard
Rename module utils to common
c9571ae 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Refactor the code to let the client, test to use the RewriteServiceProxy
a784d70 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Removed non needed classpath.tsv.gz file from spring-todo-app as the …
7582361 
…reflection is working and able to discover such files

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Add method to get the rows from the datatable object using too reflec…
b25de4a 
…tion

Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
@cmoulliard
Remobe non used import: java reflect method
79d11f3 
Signed-off-by: Charles Moulliard <cmoulliard@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cmoulliard