Skip to content

Comments

Add Records#30

Merged
Vandesm14 merged 18 commits intomainfrom
feat/records
Apr 24, 2024
Merged

Add Records#30
Vandesm14 merged 18 commits intomainfrom
feat/records

Conversation

@Vandesm14
Copy link
Owner

@Vandesm14 Vandesm14 commented Apr 23, 2024

Adds Records (objects, hash maps, etc) to Stack.

  1. Added a record type, ExprKind::Record(HashMap<Symbol, Expr>)
  2. Added intrinsic methods to interact with records
  3. Added syntax for defining records (pretty simple at the moment)
  4. Added tests for the record type and intrinsics

TODO:

  • Move the record module into intrinsics
  • Add a syntax for defining records using {}
  • Add tests

@Vandesm14 Vandesm14 marked this pull request as ready for review April 24, 2024 19:29
@Vandesm14 Vandesm14 requested a review from leonskij April 24, 2024 19:29
Copy link
Collaborator

@leonskij leonskij left a comment

Choose a reason for hiding this comment

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

On mobile, but this looks good to me.

@Vandesm14 Vandesm14 merged commit ed6c22c into main Apr 24, 2024
@Vandesm14 Vandesm14 deleted the feat/records branch April 24, 2024 21:44
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.

2 participants