Skip to content

Commit d120428

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

5 files changed

Lines changed: 2743 additions & 1073 deletions

File tree

.github/workflows/release-please.yml

Lines changed: 9 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -19,72 +19,53 @@ jobs:
1919
release-please:
2020
runs-on: ubuntu-latest
2121
steps:
22-
- name: Initialize Release Please
23-
id: release
24-
if: ${{ github.event_name == 'push' }}
25-
uses: googleapis/release-please-action@v4
26-
with:
27-
target-branch: ${{ github.ref_name }}
28-
config-file: release-please-config.json
29-
manifest-file: .release-please-manifest.json
30-
3122
- name: Checkout Sources
32-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
3323
uses: actions/checkout@v4
3424

3525
- name: Install NodeJS
36-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
3726
uses: actions/setup-node@v6
3827
with:
3928
node-version: 22
4029
registry-url: "https://registry.npmjs.org"
4130
scope: "@hpcc-js"
4231

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
32+
- name: Initialize Release Please
33+
id: release
34+
uses: googleapis/release-please-action@v4
5135
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 || '');
36+
target-branch: ${{ github.ref_name }}
37+
config-file: release-please-config.json
38+
manifest-file: .release-please-manifest.json
5539

5640
- name: Install Dependencies
57-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
5841
run: |
5942
npm ci
6043
6144
- name: Lint
62-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
6345
run: |
6446
npm run lint
6547
6648
- name: Build
67-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
6849
run: |
6950
npm run build
7051
7152
- name: Install Test Dependencies
72-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
53+
if: ${{ github.event_name == 'pull_request' }}
7354
run: |
7455
sudo apt-get update
7556
sudo npx -y playwright install chromium --with-deps
7657
npx -y playwright install chromium
7758
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
7859
7960
- name: Install HPCC Platform
80-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
61+
if: ${{ github.event_name == 'pull_request' }}
8162
continue-on-error: true
8263
run: |
8364
sudo apt install -f ./hpccsystems-platform-community_9.6.50-1jammy_amd64_withsymbols.deb
8465
sudo /etc/init.d/hpcc-init start
8566
8667
- name: Test
87-
if: ${{ github.event_name == 'pull_request' || steps.release.outputs.release_created }}
68+
if: ${{ github.event_name == 'pull_request' }}
8869
env:
8970
CI: true
9071
run: |
@@ -94,17 +75,6 @@ jobs:
9475
npm run test-node-esm
9576
npm run test-node-cjs
9677
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-
10878
- name: push stamped release
10979
if: ${{ steps.release.outputs.release_created }}
11080
run: |
@@ -121,11 +91,6 @@ jobs:
12191
run: |
12292
npm run publish
12393
124-
# - name: Purge jsdelivr
125-
# if: ${{ steps.release.outputs.release_created }}
126-
# run: |
127-
# npm run purge-jsdelivr
128-
12994
- name: Upload error logs
13095
if: ${{ failure() || cancelled() }}
13196
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)