Skip to content

Conversation

@adam-urbanczyk
Copy link
Member

@adam-urbanczyk adam-urbanczyk commented Dec 2, 2025

Updates for OCP 7.9 compatibility.

@adam-urbanczyk adam-urbanczyk marked this pull request as draft December 2, 2025 17:10
@codecov
Copy link

codecov bot commented Dec 3, 2025

Codecov Report

❌ Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.18%. Comparing base (fea85a5) to head (9b4ce19).
⚠️ Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
cadquery/occ_impl/shapes.py 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1946      +/-   ##
==========================================
+ Coverage   95.77%   96.18%   +0.40%     
==========================================
  Files          29       29              
  Lines        7856     8588     +732     
  Branches     1183     1412     +229     
==========================================
+ Hits         7524     8260     +736     
+ Misses        192      190       -2     
+ Partials      140      138       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adam-urbanczyk
Copy link
Member Author

Despite my fix 7.9 fails on cqgi and python 14. Seems to work on earlier ones.

@Oblynx
Copy link

Oblynx commented Jan 16, 2026

Hey @adam-urbanczyk I just tested this on our internal test suite, which isn't using OCP 7.9-specific features. The upgrade went well and all the tests passed. Python 3.11.

@adam-urbanczyk
Copy link
Member Author

Thanks @Oblynx , the remaining issue is 3.14 and cqgi specific.

BTW: would you be willing to describe your use case?

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.

3 participants