Closed himadri-mandal closed 1 month ago
The change in EnvironmentContextLoaderListener.java
replaces the deprecated PropertyPlaceholderConfigurer
with PropertySourcesPlaceholderConfigurer
to update the way properties are managed within the application context setup. This change ensures compatibility with Spring 5.2 and above.
File | Change Summary |
---|---|
src/main/java/ai/elimu/web/context/EnvironmentContextLoaderListener.java |
Replaced PropertyPlaceholderConfigurer with PropertySourcesPlaceholderConfigurer |
Objective | Addressed | Explanation |
---|---|---|
Replace PropertyPlaceholderConfigurer with PropertySourcesPlaceholderConfigurer (#1756) |
✅ |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 0%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 15.19%. Comparing base (
d1bd588
) to head (d9206ac
).
Files | Patch % | Lines |
---|---|---|
.../web/context/EnvironmentContextLoaderListener.java | 0.00% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks, @himadri-mandal. Did you also run
mvn jetty:run
to verify that the webapp still launches without problems?
Yes, ran mvn jetty:run
, also ran the test cases.
fix #1756