Skip to content

bump to v0.6.0, depend on cadquery-ocp-proxy 7.9.x, streamline workflows, enhance pyproject.toml, improve compat with OCP 7.9.x#46

Merged
snoyer merged 4 commits intosnoyer:mainfrom
jdegenstein:patch-3
Jan 8, 2026
Merged

Conversation

@jdegenstein
Copy link
Contributor

This is per our discussions on discord. In summary, ocpsvg needs to depend on the proxy package. I updated the package name and the version range as well as a bump to the version of this package to v0.6.0 (which is outside of the version range allowed by build123d -- that will be changed once ocp 7.9 is enabled by build123d).

@snoyer
Copy link
Owner

snoyer commented Jan 7, 2026

Thank you for figuring out the details of the upgrade!

Looks like I need to fix my outdated CI stuff first to make sure the tests still pass...

@jdegenstein
Copy link
Contributor Author

@snoyer I am happy to help with that too in this PR if you would like

@snoyer
Copy link
Owner

snoyer commented Jan 7, 2026

#47 should have done it. You'll have to rebase for it to run properly

@jdegenstein
Copy link
Contributor Author

ok @snoyer should be ready to re-run

@snoyer
Copy link
Owner

snoyer commented Jan 7, 2026

can you add the cadquery_ocp_novtk dependency to the [dev] group and do pip install .[dev] in tests.yaml and codecov.yaml?

…to [dev], replace e.g. TopoDS.Edge_s with TopoDS.Edge
@jdegenstein jdegenstein changed the title pyproject.toml -> version 0.6.0, depend on cadquery-ocp-proxy 7.9.x bump to v0.6.0, depend on cadquery-ocp-proxy 7.9.x, streamline workflows, enhance pyproject.toml, improve compat with OCP 7.9.x Jan 7, 2026
@jdegenstein
Copy link
Contributor Author

jdegenstein commented Jan 7, 2026

ok @snoyer this is ready to re-run workflows, all of them passed on my fork. lint.yaml hasn't run yet, but hopefully will pass too

EDIT: lint.yaml passed too after I added it

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.71%. Comparing base (1487bdc) to head (8399eab).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #46   +/-   ##
=======================================
  Coverage   99.71%   99.71%           
=======================================
  Files           8        8           
  Lines        1418     1418           
=======================================
  Hits         1414     1414           
  Misses          4        4           

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

@snoyer
Copy link
Owner

snoyer commented Jan 8, 2026

Looks great!

Are the *_s() static method going to stay for backwards compatibility until 8.0? If so could we make the *.py files changes in a separate PR and only keep the deps/CI stuff in this one (sorry, I didn't think of asking for that earlier)?

@snoyer snoyer merged commit 83f57aa into snoyer:main Jan 8, 2026
14 checks passed
@jdegenstein jdegenstein deleted the patch-3 branch January 16, 2026 16:07
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.

2 participants