Skip to content

Commit d5c6d95

Browse files
committed
fix(ci): fix generate SDK workflow GPG signing and push auth
- Use setup-and-install action instead of manual checkout with persist-credentials: false - Push via git push origin instead of manual URL construction - Only run cleanup-git-signing when setup-git-signing actually ran - Use checkout-ref for validate job
1 parent d07c1e6 commit d5c6d95

File tree

1 file changed

+6
-15
lines changed

1 file changed

+6
-15
lines changed

.github/workflows/generate.yml

Lines changed: 6 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,6 @@ jobs:
4545
echo "Sleeping for $delay seconds..."
4646
sleep $delay
4747
48-
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
49-
with:
50-
persist-credentials: false
51-
token: ${{ secrets.GITHUB_TOKEN }}
52-
5348
- uses: SocketDev/socket-registry/.github/actions/setup-and-install@da519693b701861bc0f9690a30f50af1a1dcd49c # main
5449

5550
- name: Generate SDK
@@ -74,12 +69,11 @@ jobs:
7469
if: steps.check.outputs.has_changes == 'true'
7570
env:
7671
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
77-
REPO: ${{ github.repository }}
7872
run: |
7973
git checkout -b automated/open-api
8074
git add .
8175
git commit -m "fix(openapi): sync with openapi definition"
82-
git push "https://x-access-token:${GH_TOKEN}@github.com/${REPO}.git" automated/open-api -fu
76+
git push origin automated/open-api -fu
8377
8478
- name: Create Pull Request
8579
if: steps.check.outputs.has_changes == 'true'
@@ -94,7 +88,7 @@ jobs:
9488
--head automated/open-api \
9589
--base main \
9690
--title "Sync with OpenAPI definition" \
97-
--body "## 🔄 OpenAPI Sync
91+
--body "## OpenAPI Sync
9892
9993
The OpenAPI definition in the API has been updated. This PR automatically:
10094
- Downloads the latest OpenAPI specification
@@ -105,15 +99,15 @@ jobs:
10599
### What's Changed
106100
See the file changes below for specific updates to the API types, strict types, and methods.
107101
108-
⚠️ **Please review carefully for any breaking changes in the API.**" \
102+
**Please review carefully for any breaking changes in the API.**" \
109103
--label "dependencies" \
110104
--label "automated"
111105
else
112106
echo "PR #$existing_pr already exists, skipping creation"
113107
fi
114108
115109
- uses: SocketDev/socket-registry/.github/actions/cleanup-git-signing@da519693b701861bc0f9690a30f50af1a1dcd49c # main
116-
if: always()
110+
if: steps.check.outputs.has_changes == 'true' && always()
117111

118112
validate:
119113
name: Validate generated SDK
@@ -125,12 +119,9 @@ jobs:
125119
outputs:
126120
has_changes: ${{ steps.check.outputs.has_changes }}
127121
steps:
128-
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
129-
with:
130-
persist-credentials: false
131-
ref: automated/open-api
132-
133122
- uses: SocketDev/socket-registry/.github/actions/setup-and-install@da519693b701861bc0f9690a30f50af1a1dcd49c # main
123+
with:
124+
checkout-ref: automated/open-api
134125

135126
- name: Build SDK
136127
run: pnpm run build

0 commit comments

Comments
 (0)