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

[DO NOT MERGE] Test branch release/2.4.x #287

Closed
wants to merge 7 commits into from

Conversation

shaobo-he-aws
Copy link
Contributor

No description provided.

shaobo-he-aws and others added 7 commits August 25, 2023 14:32
* allow `check-parse` command to read from stdin
* Adds open_attributes to the record type struct. When true, we assume a record
  with that type may have other attributes. When false it will have exactly the
  attributes specified in the type.
* Record literals and records in schema are now closed.
* Update has to use this new field. A has on a closed record will have
  type False if the attribute is not present.
The record LUB was incorrectly made Open even though the LUB attributes keys
match the input record attributes exactly. This happens only when the records
were not in a subtype relation because at least one pair of corresponding
attributes in the record were not in a subtype relation, but the pair of
attributes did have a least upper bound. For example, {a: true} and {a: false}.
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.

4 participants