Skip to content

Add Mule package type to getTopmostApplicationModules #385

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

Merged
merged 2 commits into from
Sep 18, 2022

Conversation

cab105
Copy link
Contributor

@cab105 cab105 commented Sep 6, 2022

Following up on #373, I was able to figure out why my Mule project was not being picked up by Spring Boot Migrator. Turns out there's a check in the dependent call of getTopmostApplicationModules that whitelists only jar based applications. However, Mulesoft projects built with Anypoint Studio defaults to using mule-application. Since this is the package type that I've been using for my mule migration projects, the code has consistently failed.

This PR will whitelist this application type as the migrate-mule-to-boot recipe will convert the application type to jar after it runs.

@fabapp2
Copy link
Contributor

fabapp2 commented Sep 7, 2022

Thank you @cab105 ! 🚀
I had one comment.

@cab105
Copy link
Contributor Author

cab105 commented Sep 7, 2022

Dare I ask what the one comment is?

@fabapp2
Copy link
Contributor

fabapp2 commented Sep 18, 2022

Dare I ask what the one comment is?

Sorry, my bad. 🙈

@fabapp2 fabapp2 merged commit 7bd2590 into spring-projects-experimental:main Sep 18, 2022
@cab105 cab105 deleted the whitelist-mule-app branch September 21, 2022 01:30
@fabapp2 fabapp2 added the type: enhancement New feature or request label Sep 26, 2022
@fabapp2 fabapp2 added this to the v0.12.0 milestone Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants