IntersectMBO
Get rid of redundant accessor functions
Every era defines a whole bunch of accessor functions eg, inputs', outputs', etc. which are redundant and should be removed, while all use sites should be switched to corresponding lenses. In case of ShelleyMA it even defines a pattern synonym. That one should be removed as well
In case of ShelleyMA it even defines a pattern synonym. That one should be removed as well
This part has been addressed in #3126
They should first be deprecated and then we can get rid of them at some point later. Some parts have been addressed in https://github.com/IntersectMBO/cardano-ledger/pull/4973 but there are still synonyms like that lingering around.
The only remaining undeprecated accessor functions I was able to find were in Alonzo, and I couldn't find any redundant patterns. If there are any that I've missed, please let me know.
The only remaining undeprecated accessor functions I was able to find were in Alonzo, and I couldn't find any redundant patterns.
Yep, looking for those fields to deprecate was part of this ticket. I also think that it was only in Alonzo that we had those left over. Thank you for double checking.