> If you drop self._config.get_bool("autorpt"), this reads:I'm not sure why dropping it has any relevance, but perhaps you're just
saying it reads oddly?
> + is_changed = new != prev if was_filled else TrueHow about we change it to:
> + if new and is_changed:
autorpt_enabled = self._config.get_bool("autorpt")
is_changed = new != prev if was_filled else True
if new is not None and is_changed and autorpt_enabled:
....