play-with-go / preguide

preguide is a guide/manual specification and validation tool, principally used as part of https://play-with-go.dev
BSD 3-Clause "New" or "Revised" License
11 stars 1 forks source link

cmd/preguide: cache is not invalidated by negating command #205

Closed myitcv closed 2 years ago

myitcv commented 2 years ago

If a step contains something like:

false

but is then changed to:

! false

cmd/preguide does not detect this as a cache miss, because the same bash results.

Whilst in an ideal situation we might optimise by simply ensuring the exit code from the previous run is now as expected, it might be more practical to simply cause a cache miss and re-run for now.