Skip to content

fix(ci): use correct rust-toolchain action and venv for maturin#1

Merged
farhan-syah merged 5 commits into
mainfrom
fix/ci-workflow
Nov 26, 2025
Merged

fix(ci): use correct rust-toolchain action and venv for maturin#1
farhan-syah merged 5 commits into
mainfrom
fix/ci-workflow

Conversation

@farhan-syah

Copy link
Copy Markdown
Member
  • Replace dtolnay/rust-action with dtolnay/rust-toolchain
  • Create venv for maturin develop command
  • Use .venv/bin/python for test step

- Replace dtolnay/rust-action with dtolnay/rust-toolchain
- Create venv for maturin develop command
- Use .venv/bin/python for test step
- Pin Python 3.12 in test job (runner default 3.14 exceeds PyO3 max 3.13)
- Apply rustfmt formatting fixes
- Fix clippy collapsible_if warning
- Add pre-commit hook for local development
…forms

- Consolidate clippy and rustfmt into single lint job
- Add PyO3 Python environment configuration for Ubuntu, macOS, and Windows
- Configure LIBRARY_PATH on macOS to resolve linker errors with Python framework
- Restore full Python tests with tiktoken comparison

This resolves macOS build failures caused by missing Python framework linking.
Update macOS PyO3 configuration to use the standard -undefined dynamic_lookup
linker flag for Python extension modules. Add DYLD_LIBRARY_PATH environment
variable alongside LIBRARY_PATH for proper runtime library resolution.
@farhan-syah farhan-syah merged commit d006012 into main Nov 26, 2025
5 checks passed
@farhan-syah farhan-syah deleted the fix/ci-workflow branch November 26, 2025 04:58
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.

1 participant