Skip to content

Commit 3502aeb

Browse files
author
Eyal Rozen
committed
ci: match pull request to boilerplate
1 parent 668aa7a commit 3502aeb

File tree

1 file changed

+42
-34
lines changed

1 file changed

+42
-34
lines changed

.github/workflows/pull_request.yml

Lines changed: 42 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -8,44 +8,56 @@ jobs:
88
runs-on: ubuntu-latest
99
strategy:
1010
matrix:
11-
node-version: [24.x]
11+
node: [24.x]
1212

1313
steps:
14-
- name: Check out Git repository
15-
uses: actions/checkout@v4
14+
- name: Check out TS Project Git repository
15+
uses: actions/checkout@v5
1616

17-
- name: Set up Node.js
18-
uses: actions/setup-node@v4
17+
- name: Init Nodejs
18+
uses: MapColonies/shared-workflows/actions/init-npm@init-npm-v1
1919
with:
20-
node-version: ${{ matrix.node-version }}
21-
cache: 'npm'
22-
23-
- name: Install dependencies
24-
run: npm ci
20+
node-version: ${{ matrix.node }}
2521

26-
- name: Run linters
22+
- name: Run TS Project linters
2723
uses: wearerequired/lint-action@v2
2824
with:
29-
github_token: ${{ secrets.github_token }}
3025
eslint: true
3126
prettier: true
3227
eslint_extensions: ts
28+
tsc: true
29+
openapi-lint:
30+
name: Run OpenAPI lint Check
31+
runs-on: ubuntu-latest
32+
33+
steps:
34+
- name: Check out TS Project Git repository
35+
uses: actions/checkout@v5
36+
37+
- name: Init Nodejs
38+
uses: MapColonies/shared-workflows/actions/init-npm@init-npm-v1
3339

3440
- name: OpenAPI Lint Checks
35-
uses: nwestfall/openapi-action@v1.0.2
41+
run: npx @redocly/cli lint --format=github-actions openapi3.yaml
42+
helm-lint:
43+
name: Run Helm lint Check
44+
runs-on: ubuntu-latest
45+
46+
steps:
47+
- name: Check out TS Project Git repository
48+
uses: actions/checkout@v5
3649
with:
37-
github_token: ${{ secrets.GITHUB_TOKEN }}
38-
file: ./openapi3.yaml
50+
fetch-depth: 0
51+
52+
- name: Helm Lint Checks
53+
uses: MapColonies/shared-workflows/actions/helm-lint@helm-lint-v1
3954

4055
tests:
41-
name: Run Tests (Node ${{ matrix.node-version }})
56+
name: Run Tests
4257
runs-on: ubuntu-latest
4358
strategy:
4459
matrix:
45-
node-version: [24.x]
46-
47-
container:
48-
image: node:${{ matrix.node-version }}
60+
node: [24.x]
4961

5062
services:
5163
redis:
@@ -58,10 +70,12 @@ jobs:
5870
5971
steps:
6072
- name: Check out Git repository
61-
uses: actions/checkout@v4
73+
uses: actions/checkout@v5
6274

63-
- name: Install Node.js dependencies
64-
run: npm ci
75+
- name: Init Nodejs
76+
uses: MapColonies/shared-workflows/actions/init-npm@init-npm-v1
77+
with:
78+
node-version: ${{ matrix.node }}
6579

6680
- name: Run tests
6781
run: npm run test
@@ -70,19 +84,13 @@ jobs:
7084
REDIS_HOST: redis
7185
REDIS_PORT: 6379
7286

73-
- name: Upload Artifacts
74-
uses: actions/upload-artifact@v5
75-
if: always()
87+
- uses: actions/upload-artifact@v4
7688
with:
77-
name: Test Reporters - Node v${{ matrix.node-version }}
89+
name: Test Reporters Node-${{ matrix.node }}
7890
path: reports/**
7991

80-
build_image:
81-
name: Build Image
92+
build_docker_image:
8293
runs-on: ubuntu-latest
8394
steps:
84-
- name: Check out Git repository
85-
uses: actions/checkout@v4
86-
87-
- name: build Docker image
88-
run: docker build -t test-build:latest .
95+
- name: Build and Push Docker image
96+
uses: docker/build-push-action@v6

0 commit comments

Comments
 (0)