IntersectMBO / IntersectMBO/plutus

Data.AssocMap and Data.Value should use Data.List

Open
#6,933 0 comments 0 reactions 0 assignees View on GitHub
Good first issue Internal Plinth Plutus Ledger API status: triaged
Dominant language
Haskell
Stars
1.6k
Forks
508
Avg merge
3d 10h
Merged PRs (30d)
22

Description

After https://github.com/IntersectMBO/plutus/pull/6931 is merged, you can search for uses of `toSOPList` or `unsafeFromSOPList` and you'll see that lots of code in the ledger API or the standard library does conversions to and from SOP lists which is not ideal. This is because when we implemented Data-backed Map and Value, we didn't have a Data-backed List yet. We should refactor all of these cases where a Data-backed List would be more appropriate.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.