IntersectMBO / plutus

The Plutus language implementation and tools
Apache License 2.0
1.56k stars 477 forks source link

Additions to plutus-ledger-api #5613

Closed lehins closed 10 months ago

lehins commented 10 months ago

Describe the feature you'd like

In ledger we would like to drop direct dependency on plutus-core in favor of plutus-ledger-api and avoid defining orphan instances for Plutus types. Here is a list of things that would help us do that:

Describe alternatives you've considered

No response

effectfully commented 10 months ago

See #5618.

enumerate, because we need to do the same thing in our tests

I didn't do that, because I'm not sure how sensible it is to expose an arbitrary auxiliary function from plutus-ledger-api, particularly given that it's a one-liner. I don't mind doing that, just don't know if it's right. @zliu41 @michaelpj opinions?

michaelpj commented 10 months ago

We don't even define enumerate, it's from extra.

lehins commented 10 months ago

We don't even define enumerate, it's from

Good to know. We'll use extra then :smile:

effectfully commented 10 months ago

Resolved by #5621 and #5618.