You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Per the 3rd bullet in the comment on issue 181, an additional nice feature would be yellow underlining of properties that are being overridden (or are overriding) the same property from another config source. For example, if quarkus.http.port is in both the microprofile-config.properties and application.properties, then the yellow underline message in microprofile-config.properties should be that it is being overridden by application.properties, and in application.properties, there should be a message that it is overriding the same property in microprofile-config.properties.
It would be nice to take this to the "next level", notifying of duplicate properties defined in system properties ("-D"), environmental variables, and application.yaml as well.
This requirement does not include custom or dynamic config sources, although that would admittedly be a nice touch.
The text was updated successfully, but these errors were encountered:
Per the 3rd bullet in the comment on issue 181, an additional nice feature would be yellow underlining of properties that are being overridden (or are overriding) the same property from another config source. For example, if quarkus.http.port is in both the microprofile-config.properties and application.properties, then the yellow underline message in microprofile-config.properties should be that it is being overridden by application.properties, and in application.properties, there should be a message that it is overriding the same property in microprofile-config.properties.
It would be nice to take this to the "next level", notifying of duplicate properties defined in system properties ("-D"), environmental variables, and application.yaml as well.
This requirement does not include custom or dynamic config sources, although that would admittedly be a nice touch.
The text was updated successfully, but these errors were encountered: