tweag / cooked-validators

MIT License
39 stars 11 forks source link

Make Transactions have a list of labels instead of `Maybe` a label (low priority) #164

Closed carlhammann closed 1 year ago

carlhammann commented 1 year ago

At the moment, a TxSkel can Maybe have a label attached to it, which has the intended use to show that an attack was applied. There might be a problem once we start applying many attacks to the same transaction, which is discussed in the PR introducing the monad attacks.

A possible solution would be to have a list of labels and add to that list, but only if the label does not already exist.