Skip to content

Commit

Permalink
perf(ci): Sort CI steps by length (#4243)
Browse files Browse the repository at this point in the history
* ⚡ Sort CI steps by length

* 🐛 `Build` before `test` for n8n packages
  • Loading branch information
ivov authored Oct 10, 2022
1 parent fdbc11a commit d47ff48
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci-master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ jobs:
- name: Install npm and dependencies
run: npm install -g npm@latest && npm install

- name: Lint
run: npm run lint

- name: Build
run: npm run build --if-present

- name: Test
run: npm run test

- name: Lint
run: npm run lint
12 changes: 6 additions & 6 deletions .github/workflows/ci-pull-requests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,16 +24,16 @@ jobs:
- name: Install npm and dependencies
run: npm install -g npm@latest && npm install

- name: Build
run: npm run build --if-present

- name: Test
run: npm run test

- name: Fetch base branch for `git diff`
run: git fetch origin ${{ github.event.pull_request.base.ref }}:${{ github.event.pull_request.base.ref }}

- name: Run ESLint on changes only
env:
ESLINT_PLUGIN_DIFF_COMMIT: ${{ github.event.pull_request.base.ref }}
run: npm run lint

- name: Build
run: npm run build --if-present

- name: Test
run: npm run test

0 comments on commit d47ff48

Please sign in to comment.