Skip to content

Conversation

@kl0tl
Copy link
Member

@kl0tl kl0tl commented Feb 2, 2021

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:

  • 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)

Copy link
Member

@garyb garyb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😄

@JordanMartinez
Copy link
Contributor

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

@kl0tl kl0tl mentioned this pull request Feb 17, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants