Not a merge blocker, but I think `is_enabled` is not a good name for this variable.  If `is_enabled` is `true`, it suggests it's already enabled...but really it expresses user intent to make it enabled (and it's presumably not already enabled).  I think `enable` is a better (and shorter name).