-
Improvement
-
Resolution: Unresolved
-
Minor
-
None
There are now three different before attributes for when which are evaluated in this very order:
- beforeOptions
- beforeInput
- beforeAgent
The documentation tries to describe this "highlander principle" but it would be much more logical and readable if there was only a single attribute before allowing only a single value of options, input or agent.
E.g.:
when { before git // ... }
Migration consideration: The old dedicated before attributes should not be removed right away. Instead those attributes should be marked deprecated and their usage should yield warnings in the log.
This is totally doable and would likely have minimal impact. It could even be done without changing the underlying AST representation if needed.
famod
Are you interested in submitting this change?