-
Notifications
You must be signed in to change notification settings - Fork 99
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
# [1.0.0](v0.13.0...v1.0.0) (2020-12-13) ### Bug Fixes * replace deprecated ui option with ui_config ([3830ade](3830ade)) ### BREAKING CHANGES * This cannot be updated in a non-breaking fashion, but the least disruptive route was chosen. * If both `ui` and `ui_config.enabled` are set, `ui_config` takes precedence. Hence, the change may enable the UI on machines which had previously set `ui` to false. This is arguably better than defaulting to `false`, which would disable the UI where it is supposed to be enabled. * Removing the option entirely breaks similarly if users rely on the formula defaults, since Consul's default is `false` and the formula's default used to be `true`. * The only other way to break less would be to set both options, but then users would also have override both (which is not obvious and very annoying) and there would still be no way forward to when Consul actually removes the deprecated option.
- Loading branch information
1 parent
3830ade
commit 31329c3
Showing
3 changed files
with
54 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters