The `default_enabled:` should be taken from YAML definition
Related to
Currently we have
- YAML with
default_enabled:
that provides a consistency check for all places where feature flag is used -
Feature.enabled?(..., default_enabled: ...)
that defines a desired state
This makes us to have update two places if we want to change default_enabled:
. We should instead ensure
that default_enabled:
is always taken from YAML
definition which is always loaded.
Proposal
- Remove
default_enabled:
fromFeature.enabled?
- Use
default_enabled:
from YAML