Merged
Conversation
Contributor
ianscrivener
commented
Mar 26, 2026
- STL codec bug fixes — patched issues in the STL export codec affecting reliability
- Minimum Python bumped to 3.11 — updated pyproject.toml and CI workflows; also updated black and click dependencies
- uv.lock added — full lockfile committed to pin all dependency versions
- 28 new tests added — test suite now at 623 total, with expanded coverage across STL, STEP, GLTF, GLB, SVG codecs
- New test fixtures added — cube_params_no_default.py, multi_show_object.py, and syntax_error.py for edge-case coverage
- CLI test expansion — test_cli.py grew significantly covering more command combinations and error paths
- New TECH_README.md created — 61-line technical reference document added alongside the existing README
- README overhauled — major rewrite to reflect current state of the project
jmwright
reviewed
Mar 26, 2026
jmwright
reviewed
Mar 26, 2026
jmwright
reviewed
Mar 26, 2026
jmwright
reviewed
Mar 26, 2026
| It is strongly recommended to use a Python virtual environment when installing via pip. | ||
|
|
||
| If installing on Windows, the [latest redistributable for Visual Studio](https://support.microsoft.com/en-us/help/2977003/the-latest-supported-visual-c-downloads) will need to be installed. | ||
| cq-cli is not available on PyPI, so it must be installed from source using pip and git. git must be installed for this to work. |
Member
There was a problem hiding this comment.
I should probably publish cq-cli on PyPI. I'm not sure why I have not. Maybe I'll create a new release and do that after these changes are merged.
jmwright
reviewed
Mar 26, 2026
jmwright
reviewed
Mar 26, 2026
Member
|
Thanks for all the work you have put into this @ianscrivener ! I made a few comments and approved the CI to run, which surfaced a few things (lint and tests). |
Contributor
Author
|
Done |
Member
|
@ianscrivener There are a few CI issues to fix, but I think we're close to merging. |
Contributor
Author
|
CI/CD complete... |
Member
|
@ianscrivener It seems like the |
Member
|
Thanks for this @ianscrivener . We can decide to move the CI/CD section later, if needed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.