Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Voi-16] basic nominal object support #24

Merged
merged 14 commits into from
Sep 1, 2024

Conversation

drew-y
Copy link
Collaborator

@drew-y drew-y commented Aug 23, 2024

No description provided.

Copy link

linear bot commented Aug 23, 2024

@drew-y drew-y marked this pull request as draft August 23, 2024 05:23
@drew-y drew-y marked this pull request as ready for review August 29, 2024 02:04
@drew-y drew-y marked this pull request as draft August 29, 2024 02:14
@drew-y drew-y force-pushed the drew/voi-16-basic-nominal-object-support branch from ffa8ae6 to 0347409 Compare August 31, 2024 01:08
@drew-y
Copy link
Collaborator Author

drew-y commented Aug 31, 2024

ITS WORKING :D Just gotta add some unit tests 🎉

drew-y added 4 commits August 31, 2024 16:36
We were matching nominal object initializers backwards (i.e. the type checker would approve if the nominal object had all the fields of the literal instead of the other way around).
@drew-y drew-y marked this pull request as ready for review September 1, 2024 21:47
@drew-y drew-y merged commit 9baaf9d into develop Sep 1, 2024
1 check passed
@drew-y drew-y deleted the drew/voi-16-basic-nominal-object-support branch September 9, 2024 22:35
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.

1 participant