feat: enable building as py3-none wheel - #68
Merged
Conversation
- update release CI to build wheels instead of tarballs
donn
marked this pull request as draft
August 31, 2026 12:19
Contributor
Greptile SummaryThis PR replaces native release tarballs with platform-specific Python wheels and adds a custom PEP 517 backend that packages the Silisizer executable and Tcl runtime.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| .github/workflows/release.yml | Replaces tarball publication with cibuildwheel-based Linux and macOS wheel builds, an installed-wheel smoke test, and wheel publishing. |
| wheel_build/init.py | Implements the PEP 517 backend, native build orchestration, Tcl runtime packaging, metadata generation, and checksum-verified CUDD fallback. |
| wheel_build/silisizer/main.py | Adds the installed console entry point that configures the bundled Tcl runtime and executes the packaged native binary. |
| CMakeLists.txt | Moves Tcl readline discovery into a dedicated CMake module and uses the repository-level module directory. |
| cmake/FindTclReadline.cmake | Encapsulates Tcl readline library and header discovery for native and wheel builds. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Source[Silisizer source] --> Backend[Custom PEP 517 backend]
Backend --> Native[CMake native build]
Backend --> Tcl[Tcl runtime files]
Native --> Wheel[Platform wheel]
Tcl --> Wheel
Wheel --> Test[Isolated cibuildwheel smoke test]
Test --> Artifact[Uploaded wheel artifact]
Artifact --> Release[GitHub release and download server]
Reviews (4): Last reviewed commit: "bugfix: integrity check if wheel_build i..." | Re-trigger Greptile
donn
force-pushed
the
donn/py3_none_wheel
branch
5 times, most recently
from
September 1, 2026 10:09
7271fbf to
1051c05
Compare
- ci: install darwin dependencies via homebrew (+ tap for tcl-readline) - cmake: fix module path, symlink missing module, and isolate tcl-readline finding behavior so it can be used standalone - wheel_build: build cudd from source if cannot be found by cmake
donn
force-pushed
the
donn/py3_none_wheel
branch
from
September 1, 2026 10:17
1051c05 to
1ee56b3
Compare
- add brewfile - unify how the macos setup works across ci.yml and release.yml
donn
marked this pull request as ready for review
September 2, 2026 04:24
Contributor
Author
|
@greptile-apps re-review. ignore the release race for now- only one branch does releases |
akashlevy
requested changes
Sep 2, 2026
akashlevy
approved these changes
Sep 3, 2026
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.
TODO: