Skip to content

Custom config locations should be used in bootstrap environment too #11772

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 4 commits into from
Apr 23, 2025

Conversation

msupic
Copy link
Contributor

@msupic msupic commented Apr 23, 2025

No description provided.

@msupic msupic requested a review from graemerocher April 23, 2025 11:34
@graemerocher graemerocher added the type: improvement A minor improvement to an existing feature label Apr 23, 2025
msupic added 3 commits April 23, 2025 15:22
…nfig locations so the array from the builder is not updated when Collections.reverse method is called on the list returned from getOverrideConfigLocations
… DefaultEnvironment constructor to create a new list before executing Collections.reverse
Copy link

@graemerocher graemerocher merged commit 956f38f into 4.8.x Apr 23, 2025
22 checks passed
@graemerocher graemerocher deleted the custom-config-fix branch April 23, 2025 15:36
msupic added a commit that referenced this pull request Apr 28, 2025
…11772)

* Custom config locations should be used in bootstrap environment too

* Modified getOverrideConfigLocations method to return a new list of config locations so the array from the builder is not updated when Collections.reverse method is called on the list returned from getOverrideConfigLocations

* Rolled back changes in getOverrideConfigLocations method and modified DefaultEnvironment constructor to create a new list before executing Collections.reverse

* Code cleanup
altro3 pushed a commit to altro3/micronaut-core that referenced this pull request Jun 3, 2025
…icronaut-projects#11772)

* Custom config locations should be used in bootstrap environment too

* Modified getOverrideConfigLocations method to return a new list of config locations so the array from the builder is not updated when Collections.reverse method is called on the list returned from getOverrideConfigLocations

* Rolled back changes in getOverrideConfigLocations method and modified DefaultEnvironment constructor to create a new list before executing Collections.reverse

* Code cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: improvement A minor improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants