abollini commented on PR #425: URL: https://github.com/apache/commons-configuration/pull/425#issuecomment-2127239110
hi @garydgregory thanks for looking to that. I have updated the PR to flag the variable as final as by your suggestion. I confirm that the expectation is to have preserved the goal of the fix for CONFIGURATION-834 Indeed, the test introduced by @kbarlowgw is still there https://github.com/apache/commons-configuration/pull/425/files#diff-b3d5f2d39a771e588b33d0d9719c4edcbbdaf3059bca13b4f70d7e0226d74ee8R106 I have only updated the test value to make the test more reliable, moreover in the test that I have added for the multiple values the interpolation is also checked https://github.com/apache/commons-configuration/pull/425/files#diff-b3d5f2d39a771e588b33d0d9719c4edcbbdaf3059bca13b4f70d7e0226d74ee8R115 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
