-
Notifications
You must be signed in to change notification settings - Fork 0
Epic/pysof #8
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
base: main
Are you sure you want to change the base?
Epic/pysof #8
Conversation
* feat/uv: complete MVP packaging docs and setup * feat: add ci for python build * feat: move the ci for python * fix: update lock * fix: update readme for macos and runners selfhosted * tech debt: remove CI momentarly
* feat: first iteration of bindings * feat: first binding completed * feat: switch build to maturin * feat: complete the core functions * feat: add tests for content types * docs: update readme for v1 * feat: multiple engine support
* feat: add pipe and script to build wheels locally * feat:split pythonc ci steps * fix:remove rust step * fix: orchestrate workflow for ci at python * fix: workflow release, removed PyPI for a bit * tech debt: remove windows build temp * fix: unify setup venv * fix: split python ci/cd * fix: remove deprecated pipes * fix: optimize venv use * fix: venv path * fix: test coverage * fix: add maturin * fix: install maturin? * fix: add rust toolchain * fix: linter issues * fix: linter target path * fix: path for formats and linters * fix: lint and checker target * fix: last iter * fix: args paths * checkpoint 1
* feat: enable wheel artifact * fix: remove restrictive branches, trigger on py-CI finish * checkpoint? * checkpoint pr * Revert "checkpoint pr" This reverts commit b6828d1. * fix:add feture branch for testing * change if for trigger
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
* feat: enable all feature on release * fix: release args --------- Co-authored-by: dougc95 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good with these comments. If someone clones this repo, but doesn't yet have uv installed, will the rust cargo build still succeed ok?
I've updated to exclude it and added a note on how to build it at |
…r serde, serde_json and chrono
…heel build from CI
…ith dtolnay/rust-toolchain
No description provided.