Remove Record.ST.new - Githubissues

purescript / purescript-record

Functions for working with records and polymorphic labels

BSD 3-Clause "New" or "Revised" License

70 stars 31 forks source link

Remove Record.ST.new #77

Closed kl0tl closed 3 years ago

kl0tl commented 3 years ago

Description of the change

There’s no way to add fields to a mutable record, this would require the ST monad to track the added fields with row indices, so Record.ST.new can only be frozen with Record.ST.freeze or Record.ST.run, which is lot less convenient than {} 😅

Checklist:

[x] Added the change to the changelog's "Unreleased" section with a reference to this PR (e.g. "- Made a change (#0000)")
[ ] Linked any existing issues or proposals that this pull request should close
[ ] Updated or added relevant documentation
[ ] Added a test for the contribution (if applicable)

JordanMartinez commented 3 years ago

Should this PR just be updated to remove the ST module completely?