Skip to content

Commit d175704

Browse files
Inerskaclaude
andcommitted
fix(ci): remove invalid napi flag and redundant test step
- Remove --gh-release=false (invalid syntax, just omit the flag) - Remove test step from publish-facetpack (already runs in test job) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent f7fb9ef commit d175704

2 files changed

Lines changed: 2 additions & 5 deletions

File tree

.github/workflows/publish.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -336,10 +336,7 @@ jobs:
336336
run: bun run build
337337
working-directory: packages/facetpack
338338

339-
- name: Test facetpack
340-
run: bun test
341-
working-directory: packages/facetpack
342-
339+
# Tests already run in the 'test' job, skip here to avoid needing binary
343340
- name: Publish facetpack
344341
run: npm publish --provenance --access public
345342
working-directory: packages/facetpack

packages/facetpack-native/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
"format:toml": "taplo format",
6262
"format:rs": "cargo fmt",
6363
"lint": "oxlint .",
64-
"prepublishOnly": "napi prepublish -t npm --skip-optional-publish --gh-release=false",
64+
"prepublishOnly": "napi prepublish -t npm --skip-optional-publish",
6565
"test": "ava",
6666
"test:coverage": "c8 ava",
6767
"test:rust": "cargo test",

0 commit comments

Comments
 (0)