-
Notifications
You must be signed in to change notification settings - Fork 182
Closed
Labels
current-sprintBeing worked on in the current sprintBeing worked on in the current sprintgood first issueA good issue to start working on Chalk withA good issue to start working on Chalk with
Description
Currently we have the ProgramClause type, which maps to an enum ProgramClauseData with two options:
- Implies -- a
ProgramClauseImplication, corresponding toP :- Q1..Qn(i.e.,Pis true ifQ1...Qnare provable) - ForAll -- which wraps the same
ProgramClauseImplicationin binders, corresponding toforall<...> { P :- Q1..qn }.
I think we should remove the Implies variant and just have that a ProgramClause maps to a Binders<ProgramClauseImplication>, but those binders may be empty.
This is a straight refactoring and simplification.
This issue has been assigned to @declanvk via this comment.
Metadata
Metadata
Assignees
Labels
current-sprintBeing worked on in the current sprintBeing worked on in the current sprintgood first issueA good issue to start working on Chalk withA good issue to start working on Chalk with