bundlewrap
bundlewrap copied to clipboard
`bw verify` should check `unless` conditions of `action` items
I just did a bw verify against one node, which didn't show any pending changes. However, when doing an bw apply -i, it prompted me to run an action, because its unless failed.
bw verify should run unless to show the user which actions would be executed.
It should be safe to implement this, because the docs say:
The commands you choose for
unlessshould not change the state of your node. Otherwise, runningbw verifymight unexpectedly interfere with your nodes.
However, we do have at least two places in our repo where we break this assumption.
Changing this could be surprising for users. Should probably be done in 5.0. 🤔