Linting: show a soft warning if variables are overridden with default value
fahad19 opened this issue · comments
Fahad Heylaal commented
Currently
- We have linting for checking entity shapes: https://featurevisor.com/docs/linting/
- Features can have variables schema with their default values defined: https://featurevisor.com/docs/features/#variables
- Those variables can be overridden too: https://featurevisor.com/docs/features/#overriding-variables
Variables always fall back to their default value, and do not need overriding if that's the value you desire for e.g. at rule level.
Proposal
Throw a soft warning (without exiting with an error code) when running $ featurevisor lint
if variables are overridden with their same default value, because it is redundant.
Where?
https://github.com/featurevisor/featurevisor/tree/main/packages/core/src/linter