Skip to content
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

Fix check for springBootApplication elements #464

Open
cherylking opened this issue Nov 20, 2024 · 0 comments
Open

Fix check for springBootApplication elements #464

cherylking opened this issue Nov 20, 2024 · 0 comments
Assignees

Comments

@cherylking
Copy link
Member

The check added recently for more than one springBootApplication element only considered a single server config file. The check needs to take into account all server config files (via include elements and configDropins). I suggest the code should collect a Map of springBootApplication by id and preserve the correct merge/replace behavior based on where the config resides. Once all the server config files have been processed, check if more than one configured springBootApplication location exists and throw the error at that point. We could even include the ids for the elements we found in the message.

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

No branches or pull requests

2 participants