Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve workflows #175

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
22 changes: 0 additions & 22 deletions .github/workflows/lint.yml

This file was deleted.

18 changes: 0 additions & 18 deletions .github/workflows/macos.yml

This file was deleted.

77 changes: 63 additions & 14 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,31 +2,80 @@ name: linux

on:
push:
branches: '*'
branches:
- main
pull_request:
branches: '*'

jobs:
test:
lint:
runs-on: ubuntu-latest
name: Lint
steps:
- name: Checkout repo
uses: actions/checkout@v4
with:
persist-credentials: false
- name: Setup node
uses: actions/setup-node@v4
with:
node-version: lts/*
- name: Install Dependencies
run: npm install
- name: Run lint
run: npm run lint

test-alpine:
runs-on: ubuntu-latest
name: Test alpine
steps:
- name: Checkout repo
uses: actions/checkout@v4
with:
persist-credentials: false
- run: |
docker build . -t pidusage
docker run -v $(pwd):/var/pidusage pidusage:latest npm install
docker run -v $(pwd):/var/pidusage pidusage:latest npm test

test:
name: Test with Node ${{ matrix.node }} on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
strategy:
matrix:
node: [ '10', '12', '14', '16' ]
name: Test Node ${{ matrix.node }}
os:
- ubuntu-latest
- windows-latest
- macos-latest
node:
- 10
- 12
- 14
- 16
- 18
- 20
- 21
steps:
- uses: actions/checkout@v2
- name: Setup node
uses: actions/setup-node@v2
- name: Checkout repo
uses: actions/checkout@v4
with:
persist-credentials: false
- name: Setup node ${{ matrix.node }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
- run: npm install
- run: npm test
- name: Install Dependencies
run: npm install
- name: Run tests
run: npm test

coverage:
runs-on: ubuntu-latest
name: Test coverage
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Setup node
uses: actions/setup-node@v2
- run: npm install
- run: npm run coverage
uses: actions/setup-node@v4
- name: Install Dependencies
run: npm install
- name: Run coverage
run: npm run coverage
18 changes: 0 additions & 18 deletions .github/workflows/windows.yml

This file was deleted.