Skip to content

Commit 6078c7a

Browse files
kalwaltclaude
andcommitted
fix(ci): harden publish-crates checkout against submodule HTTP 500s
Follow-up to #80 (which closed #79). After merging #80, the v0.3.5 tag push retriggered release.yml. The `release` job succeeded (checkout@v6 cloned the WebARKitLib submodule fine), but `publish-crates` failed twice in a row with: error: RPC failed; HTTP 500 fatal: expected 'packfile' fatal: clone of 'https://github.com/webarkit/WebARKitLib.git' into submodule path '...WebARKitLib' failed GitHub retried once internally; the retry also got HTTP 500. Net result: crates.io still shipped no tarball, so #79 is effectively still open for crates.io consumers. Two changes to publish-crates (release and publish-npm left alone — release already works, publish-npm doesn't need submodules): - Bump actions/checkout@v5 -> @v6. The release job on the same run used v6 and cloned successfully against the same submodule endpoint; the only meaningful difference from the failing publish-crates job was the checkout major version. - Add fetch-depth: 0. Removes --depth=1 from the submodule clone. The WebARKitLib repo is ~3.8MB, so full history is effectively free. Smart-http shallow-clone negotiation has known edge cases that can surface as HTTP 500; a full clone sidesteps that code path entirely. The `Verify vendored sources` step remains — if the submodule ever regresses to empty, it fails loudly before cargo publish is reached. Post-merge action: bump patch (0.3.6) and push the tag to retrigger publish-crates against this hardened checkout. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 0976c91 commit 6078c7a

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

.github/workflows/release.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,10 @@ jobs:
6262
needs: [release]
6363
runs-on: ubuntu-latest
6464
steps:
65-
- uses: actions/checkout@v5
65+
- uses: actions/checkout@v6
6666
with:
6767
submodules: recursive
68+
fetch-depth: 0
6869
- name: Install Rust
6970
uses: dtolnay/rust-toolchain@stable
7071
- name: Verify vendored sources

0 commit comments

Comments
 (0)