Skip to content

Commit 3213c34

Browse files
committed
fix: remove "tmp" dependency
Trivial security issue Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
1 parent aa1ccc8 commit 3213c34

5 files changed

Lines changed: 2741 additions & 1070 deletions

File tree

.github/workflows/release-please.yml

Lines changed: 7 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -19,72 +19,54 @@ jobs:
1919
release-please:
2020
runs-on: ubuntu-latest
2121
steps:
22+
- name: Checkout Sources
23+
uses: actions/checkout@v4
24+
2225
- name: Initialize Release Please
2326
id: release
24-
if: ${{ github.event_name == 'push' }}
2527
uses: googleapis/release-please-action@v4
2628
with:
29+
token: ${{ secrets.GITHUB_TOKEN }}
2730
target-branch: ${{ github.ref_name }}
2831
config-file: release-please-config.json
2932
manifest-file: .release-please-manifest.json
3033

31-
- name: Checkout Sources
32-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
33-
uses: actions/checkout@v4
34-
3534
- name: Install NodeJS
36-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
3735
uses: actions/setup-node@v6
3836
with:
3937
node-version: 22
4038
registry-url: "https://registry.npmjs.org"
4139
scope: "@hpcc-js"
4240

43-
- name: Install OS Dependencies
44-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
45-
run: |
46-
pip install pandas scikit-learn
47-
48-
- name: Export GitHub Actions cache environment variables
49-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
50-
uses: actions/github-script@v7
51-
with:
52-
script: |
53-
core.exportVariable('ACTIONS_CACHE_URL', process.env.ACTIONS_CACHE_URL || '');
54-
core.exportVariable('ACTIONS_RUNTIME_TOKEN', process.env.ACTIONS_RUNTIME_TOKEN || '');
55-
5641
- name: Install Dependencies
57-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
5842
run: |
5943
npm ci
6044
6145
- name: Lint
62-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
6346
run: |
6447
npm run lint
6548
6649
- name: Build
67-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
6850
run: |
6951
npm run build
7052
7153
- name: Install Test Dependencies
72-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
54+
if: ${{ github.event_name == 'pull_request' }}
7355
run: |
7456
sudo apt-get update
7557
sudo npx -y playwright install chromium --with-deps
7658
npx -y playwright install chromium
7759
wget https://github.com/hpcc-systems/HPCC-Platform/releases/download/community_9.6.50-1/hpccsystems-platform-community_9.6.50-1jammy_amd64_withsymbols.deb
7860
7961
- name: Install HPCC Platform
80-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
62+
if: ${{ github.event_name == 'pull_request' }}
8163
continue-on-error: true
8264
run: |
8365
sudo apt install -f ./hpccsystems-platform-community_9.6.50-1jammy_amd64_withsymbols.deb
8466
sudo /etc/init.d/hpcc-init start
8567
8668
- name: Test
87-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
69+
if: ${{ github.event_name == 'pull_request' }}
8870
env:
8971
CI: true
9072
run: |
@@ -94,17 +76,6 @@ jobs:
9476
npm run test-node-esm
9577
npm run test-node-cjs
9678
97-
# - name: Calculate Coverage
98-
# if: ${{ steps.release.outputs.release_created }}
99-
# run: |
100-
# npm run coverage
101-
102-
# - name: Update Coveralls
103-
# if: ${{ steps.release.outputs.release_created }}
104-
# uses: coverallsapp/github-action@v2
105-
# with:
106-
# github-token: ${{ secrets.GITHUB_TOKEN }}
107-
10879
- name: push stamped release
10980
if: ${{ steps.release.outputs.release_created }}
11081
run: |
@@ -121,11 +92,6 @@ jobs:
12192
run: |
12293
npm run publish
12394
124-
# - name: Purge jsdelivr
125-
# if: ${{ steps.release.outputs.release_created }}
126-
# run: |
127-
# npm run purge-jsdelivr
128-
12995
- name: Upload error logs
13096
if: ${{ failure() || cancelled() }}
13197
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)