Skip to content

Commit

Permalink
Merge pull request #5 from samzong/fmt/workflow
Browse files Browse the repository at this point in the history
fmt: format workflow and upgrade node version to 20
  • Loading branch information
samzong authored Feb 1, 2024
2 parents 67f4201 + 4e40001 commit f0aeed3
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 57 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 18
node-version: 20
cache: yarn

- name: Install dependencies
Expand Down
51 changes: 0 additions & 51 deletions .github/workflows/test-loop-action.yml

This file was deleted.

40 changes: 35 additions & 5 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Test deployment
name: Test and label PRs

on:
pull_request:
Expand All @@ -8,17 +8,47 @@ on:
# https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#on

jobs:
test-deploy:
name: Test deployment
# PR test
pr-test:
name: PR test
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 18
node-version: 20
cache: yarn

- name: Install dependencies
run: yarn install --frozen-lockfile
- name: Test build website
run: yarn build
run: yarn build

# add pull request size label
size-labeler:
permissions:
issues: write
pull-requests: write
runs-on: ubuntu-latest
name: Label the PR size
steps:
- uses: codelytv/pr-size-labeler@v1
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
xs_label: "size/xs"
xs_max_size: "9"
s_label: "size/s"
s_max_size: "29"
m_label: "size/m"
m_max_size: "99"
l_label: "size/l"
l_max_size: "499"
xl_label: "size/xl"
xl_max_size: "999"
xll_label: "size/xll"
xll_max_size: "1999"
fail_if_xll: "false"
message_if_xll: >
This PR exceeds the recommended size of 2000 lines.
Please make sure you are NOT addressing multiple issues with one PR.
Note this PR might be rejected due to its size.

0 comments on commit f0aeed3

Please sign in to comment.