Skip to content

fix: move main entrypoint class to com.embabel.template package#39

Closed
sivaprasadreddy wants to merge 1 commit intoembabel:mainfrom
sivaprasadreddy:move-entrypoint-class
Closed

fix: move main entrypoint class to com.embabel.template package#39
sivaprasadreddy wants to merge 1 commit intoembabel:mainfrom
sivaprasadreddy:move-entrypoint-class

Conversation

@sivaprasadreddy
Copy link
Contributor

No description provided.

@johnsonr
Copy link
Contributor

This was how the template originally was. While it makes sense from a Spring Boot perspective, it means that if the user adds a package parallel to template it won't be picked up. I wonder if that's something people might try first.

Of course the user might add packages in their own namespace (non embabel) which would require moving the class.

@sivaprasadreddy
Copy link
Contributor Author

Considering Embabel built on-top of Spring AI, most of the people follow Spring Boot conventions. So, I think the generated code should align with that convention as a good starting point.

@johnsonr
Copy link
Contributor

Agree. Have made this change

@johnsonr johnsonr closed this Sep 27, 2025
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.

2 participants