Skip to content

Commit b5d2dfb

Browse files
authored
Engineering - more github action cleanup (microsoft#255987)
1 parent 5f9c70a commit b5d2dfb

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

.github/workflows/pr-darwin-test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ jobs:
6868
VSCODE_ARCH: ${{ env.VSCODE_ARCH }}
6969
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
7070
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
71-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
71+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
7272
# Avoid using dlopen to load Kerberos on macOS which can cause missing libraries
7373
# https://github.com/mongodb-js/kerberos/commit/04044d2814ad1d01e77f1ce87f26b03d86692cf2
7474
# flipped the default to support legacy linux distros which shouldn't happen
@@ -101,7 +101,7 @@ jobs:
101101
if: steps.cache-builtin-extensions.outputs.cache-hit != 'true'
102102
run: node build/lib/builtInExtensions.js
103103
env:
104-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
104+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
105105

106106
- name: Transpile client and extensions
107107
run: npm run gulp transpile-client-esbuild transpile-extensions
@@ -125,7 +125,7 @@ jobs:
125125
sleep 5 # optional: add a small delay between retries
126126
done
127127
env:
128-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
128+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
129129

130130
- name: 🧪 Run unit tests (Electron)
131131
if: ${{ inputs.electron_tests }}

.github/workflows/pr-linux-test.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ jobs:
7979
echo "Npm install failed $i, trying again..."
8080
done
8181
env:
82-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
82+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
8383

8484
- name: Install dependencies
8585
if: steps.cache-node-modules.outputs.cache-hit != 'true'
@@ -101,7 +101,7 @@ jobs:
101101
VSCODE_ARCH: ${{ env.VSCODE_ARCH }}
102102
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
103103
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
104-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
104+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
105105

106106
- name: Create node_modules archive
107107
if: steps.cache-node-modules.outputs.cache-hit != 'true'
@@ -129,7 +129,7 @@ jobs:
129129
if: steps.cache-builtin-extensions.outputs.cache-hit != 'true'
130130
run: node build/lib/builtInExtensions.js
131131
env:
132-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
132+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
133133

134134
- name: Transpile client and extensions
135135
run: npm run gulp transpile-client-esbuild transpile-extensions
@@ -153,7 +153,7 @@ jobs:
153153
sleep 5 # optional: add a small delay between retries
154154
done
155155
env:
156-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
156+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
157157

158158
- name: 🧪 Run unit tests (Electron)
159159
if: ${{ inputs.electron_tests }}

.github/workflows/pr-win32-test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ jobs:
7878
VSCODE_ARCH: ${{ env.VSCODE_ARCH }}
7979
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
8080
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
81-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
81+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
8282

8383
- name: Create node_modules archive
8484
if: steps.node-modules-cache.outputs.cache-hit != 'true'
@@ -110,7 +110,7 @@ jobs:
110110
if: steps.cache-builtin-extensions.outputs.cache-hit != 'true'
111111
run: node build/lib/builtInExtensions.js
112112
env:
113-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
113+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
114114

115115
- name: Transpile client and extensions
116116
shell: pwsh
@@ -134,7 +134,7 @@ jobs:
134134
}
135135
}
136136
env:
137-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
137+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
138138

139139
- name: 🧪 Run unit tests (Electron)
140140
if: ${{ inputs.electron_tests }}

.github/workflows/pr.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ jobs:
6464
env:
6565
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
6666
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
67-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
67+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
6868

6969
- name: Create node_modules archive
7070
if: steps.cache-node-modules.outputs.cache-hit != 'true'
@@ -84,7 +84,7 @@ jobs:
8484
- name: Compile & Hygiene
8585
run: npm exec -- npm-run-all -lp core-ci-pr extensions-ci-pr hygiene eslint valid-layers-check define-class-fields-check vscode-dts-compile-check tsec-compile-check
8686
env:
87-
GITHUB_TOKEN: ${{ github.event.pull_request.head.repo.full_name == github.repository && secrets.VSCODE_OSS || secrets.GITHUB_TOKEN }}
87+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
8888

8989
linux-cli-tests:
9090
name: Linux

0 commit comments

Comments
 (0)