Skip to content

Commit 923999a

Browse files
starpitclaude
andcommitted
fix: publish spnl-ffi before spnl in crate publish workflow
spnl has an optional dependency on spnl-ffi, but cargo publish validates all dependencies exist on crates.io even when optional. Adding the missing publish step for spnl-ffi fixes the upload failure. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Nick Mitchell <nickm@us.ibm.com>
1 parent 27a2797 commit 923999a

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/publish_crate.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,12 +41,13 @@ jobs:
4141
# Users can enable CUDA features explicitly when needed
4242
run: |
4343
cargo publish --manifest-path crates/spnl-core/Cargo.toml $DRY_RUN
44-
# spnl-run and spnl depend on spnl-core being on crates.io, so
45-
# dry-run (PR) can only validate spnl-core; real publish runs all three.
44+
# spnl-run, spnl-ffi, and spnl depend on spnl-core being on crates.io, so
45+
# dry-run (PR) can only validate spnl-core; real publish runs all.
4646
if [ "$DRY_RUN" = " " ]; then
4747
sleep 30 # wait for crates.io to index spnl-core
4848
cargo publish --manifest-path crates/spnl-run/Cargo.toml
49-
sleep 30 # wait for crates.io to index spnl-run
49+
cargo publish --manifest-path crates/spnl-ffi/Cargo.toml
50+
sleep 30 # wait for crates.io to index spnl-run and spnl-ffi
5051
cargo publish --manifest-path crates/spnl/Cargo.toml
5152
fi
5253
env:

0 commit comments

Comments
 (0)