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

Misc patches leading to the term model #1343

Merged
merged 6 commits into from
Jun 16, 2021
Merged

Misc patches leading to the term model #1343

merged 6 commits into from
Jun 16, 2021

Conversation

robdockins
Copy link
Contributor

A variety of minor changes and fixes required by the term model, or discovered while working on it.

@robdockins robdockins requested a review from brianhuffman June 16, 2021 20:41
@brianhuffman
Copy link
Contributor

The new term defaulting functionality might be useful for #317.

…pes.

The term model evaluates values in the environment when reflecting
neutral terms, so we can't put a panic here.  We could also probably
be more careful about evaluating only environment terms that are
actually needed, but this is an easy fix.
to external constants, but first eta-collapsing as much
as possible.
@robdockins robdockins added the PR: ready to merge Magic flag for pull requests to ask Mergify to merge given an approval and a successful CI run label Jun 16, 2021
@mergify mergify bot merged commit dabdee6 into master Jun 16, 2021
@mergify mergify bot deleted the misc branch June 16, 2021 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready to merge Magic flag for pull requests to ask Mergify to merge given an approval and a successful CI run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants