kalekundert / stepwise

Modular, command-line scientific protocols
GNU General Public License v3.0
3 stars 0 forks source link

Make `stash pop` hide rather than delete #31

Closed kalekundert closed 4 years ago

kalekundert commented 4 years ago

I've found that I can really never use sw stash pop, because I'm never sure that something won't go wrong (e.g. the printer somehow doesn't work) that wouldn't cause me to need to view the same protocol again. So instead I only use peek, and end up with the stash being filled with protocols I've already done.

The pop command is really meant to communicate that, "I'm doing this protocol now, no need to show it in the stash anymore". I think a better/safer way to achieve this is to have pop hide the protocol rather than delete it. After being popped, the protocol would keep its id and could still be referenced by commands like peek and pop, it just wouldn't be displayed by default.

Some related changes:

kalekundert commented 4 years ago

Fixd by 58e93b9