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

Add "Fixed" to ConstraintKinds type definition #1177

Merged

Conversation

sethfischer
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Oct 10, 2022

Codecov Report

Merging #1177 (14cbdc4) into master (083e5da) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            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              
Impacted Files Coverage Δ
cadquery/assembly.py 96.53% <100.00%> (-0.02%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jmwright
Copy link
Member

@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.

Copy link
Member

@lorenzncode lorenzncode left a 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.

@sethfischer sethfischer force-pushed the typing-add-fixed-to-constraint-kinds branch from 2af2f02 to 1f35de8 Compare October 11, 2022 06:30
@jmwright
Copy link
Member

@sethfischer It looks like the black lint check is failing.

@sethfischer sethfischer force-pushed the typing-add-fixed-to-constraint-kinds branch from 1f35de8 to a252c2a Compare October 11, 2022 09:53
@adam-urbanczyk
Copy link
Member

LGTM if CI passes.

@lorenzncode
Copy link
Member

Thanks @sethfischer! FYI, the black check requires version 19.10b0.

@lorenzncode lorenzncode self-requested a review October 12, 2022 01:26
@jmwright jmwright merged commit 8ddad3b into CadQuery:master Oct 12, 2022
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