Skip to content

Commit

Permalink
build: fix release bug (#316)
Browse files Browse the repository at this point in the history
  • Loading branch information
giladgd authored Sep 18, 2024
1 parent d4a4284 commit 253b5d6
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,9 @@ jobs:
name: llama.cpp
path: llama
- name: Apply fix patch on semantic-release, to not check for push permission on dry run
run: git apply --ignore-whitespace ./scripts/patches/semantic-release+24.1.1.patch
run: |
git apply --ignore-whitespace ./scripts/patches/semantic-release+24.1.1.patch
git apply --ignore-whitespace ./scripts/patches/@semantic-release+npm+12.0.1.patch
- name: Resolve next release
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down
2 changes: 2 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@
"@nolebase/vitepress-plugin-og-image": "^2.5.0",
"@resvg/resvg-js": "^2.6.2",
"@semantic-release/exec": "^6.0.3",
"@semantic-release/npm": "12.0.1",
"@shikijs/vitepress-twoslash": "^1.17.7",
"@types/async-retry": "^1.4.8",
"@types/bytes": "^3.1.4",
Expand Down
15 changes: 15 additions & 0 deletions scripts/patches/@semantic-release+npm+12.0.1.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
diff --git a/node_modules/@semantic-release/npm/lib/verify-auth.js b/node_modules/@semantic-release/npm/lib/verify-auth.js
index 99e138e..31dee5f 100644
--- a/node_modules/@semantic-release/npm/lib/verify-auth.js
+++ b/node_modules/@semantic-release/npm/lib/verify-auth.js
@@ -12,6 +12,10 @@ export default async function (npmrc, pkg, context) {
stdout,
stderr,
} = context;
+
+ if (context.options?.dryRun)
+ return;
+
const registry = getRegistry(pkg, context);

await setNpmrcAuth(npmrc, registry, context);

0 comments on commit 253b5d6

Please sign in to comment.