-
Notifications
You must be signed in to change notification settings - Fork 299
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
Add "Fixed" to ConstraintKinds type definition #1177
Add "Fixed" to ConstraintKinds type definition #1177
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1177 +/- ##
==========================================
- Coverage 94.01% 94.01% -0.01%
==========================================
Files 26 26
Lines 5411 5410 -1
Branches 918 918
==========================================
- Hits 5087 5086 -1
Misses 193 193
Partials 131 131
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@adam-urbanczyk @lorenzncode I feel like you two are best suited to review constraints PRs, so I went ahead and added you both as reviewers on this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest to add ConstraintKind
to the imports from occ_impl.solver
along with the already imported UnaryConstraintKind, BinaryConstraintKind.
2af2f02
to
1f35de8
Compare
@sethfischer It looks like the black lint check is failing. |
1f35de8
to
a252c2a
Compare
LGTM if CI passes. |
Thanks @sethfischer! FYI, the black check requires version 19.10b0. |
No description provided.