This PR moves default value descriptions to "description" properties in logging property metadata as it doesn't seem to be conventional.
Feel free to close this if it's intentional for some reason.
See gh-41933
Comment From: wilkinsona
Thanks, @izeye, but this is intentional. The default value in the metadata is equivalent to the value to which a field is initialized in a @ConfigurationProperties class. As such, it shouldn't be descriptive. It should just be the actual default value. In this case, there isn't such a value so we have to add something to the description explaining the situation.
Comment From: izeye
@wilkinsona Thanks for the feedback!
In this case, there isn't such a value so we have to add something to the description explaining the situation.
I'm a bit confused with this as it sounds like the description is added to the description property, not the defaultValue property.
You mean "the defaultValue explaining the situation", not "the description explaining the situation", right?
Comment From: wilkinsona
Oh dear, I somehow managed to read the diff backwards. Thanks for persevering with me. My comment above is, in fact, an explanation of why your change is needed 😬.
Comment From: mhalbritter
Thanks @izeye !