Skip to content

Commit

Permalink
Update for pino@9 (#87)
Browse files Browse the repository at this point in the history
* replace husky with @fastify/pre-commit

* update ci

* remove tap

* fix broken test

* maybe?

* maybe?

* remove coveralls

* just use await completed for everything

* remove Node v16

* Windows, sigh
  • Loading branch information
jsumners committed Apr 6, 2024
1 parent a209620 commit 2cdc122
Show file tree
Hide file tree
Showing 6 changed files with 102 additions and 105 deletions.
21 changes: 2 additions & 19 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ jobs:
contents: read
strategy:
matrix:
node-version: [14, 16, 18]
node-version: [18, 20]
os: [macos-latest, ubuntu-latest, windows-latest]
exclude:
- node-version: 14
Expand All @@ -59,30 +59,13 @@ jobs:
with:
path: node_modules
key: node-modules-${{ hashFiles('package.json') }}

- name: Install dependencies
run: npm i --ignore-scripts

- name: Run Tests
run: npm run test-ci

- name: Coveralls Parallel
uses: coverallsapp/github-action@v2.1.2
with:
github-token: ${{ secrets.github_token }}
parallel: true
flag-name: run-${{ matrix.node-version }}-${{ matrix.os }}

coverage:
needs: test
runs-on: ubuntu-latest
steps:
- name: Coveralls Finished
uses: coverallsapp/github-action@v2.1.2
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
parallel-finished: true

automerge:
name: Automerge Dependabot PRs
if: >
Expand Down
1 change: 0 additions & 1 deletion .husky/.gitignore

This file was deleted.

4 changes: 0 additions & 4 deletions .husky/pre-commit

This file was deleted.

1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# pino-abstract-transport
[![npm version](https://img.shields.io/npm/v/pino-abstract-transport)](https://www.npmjs.com/package/pino-abstract-transport)
[![Build Status](https://img.shields.io/github/workflow/status/pinojs/pino-abstract-transport/CI)](https://github.com/pinojs/pino-abstract-transport/actions)
[![Coverage Status](https://coveralls.io/repos/github/pinojs/pino-abstract-transport/badge.svg?branch=master)](https://coveralls.io/github/pinojs/pino-abstract-transport?branch=master)
[![js-standard-style](https://img.shields.io/badge/code%20style-standard-brightgreen.svg?style=flat)](https://standardjs.com/)

Write Pino transports easily.
Expand Down
15 changes: 9 additions & 6 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,8 @@
"description": "Write Pino transports easily",
"main": "index.js",
"scripts": {
"prepare": "husky install",
"test": "standard | snazzy && tap test/*.test.js && tsd",
"test-ci": "standard | snazzy && tap test/*.test.js --coverage-report=lcovonly && tsd"
"test": "standard | snazzy && c8 node --test test/base.test.js && tsd",
"test-ci": "npm test"
},
"repository": {
"type": "git",
Expand All @@ -27,14 +26,18 @@
"split2": "^4.0.0"
},
"devDependencies": {
"@fastify/pre-commit": "^2.1.0",
"@matteo.collina/tspl": "^0.1.1",
"@types/node": "^20.1.0",
"husky": "^9.0.6",
"c8": "^9.1.0",
"snazzy": "^9.0.0",
"standard": "^17.0.0",
"tap": "^16.0.0",
"tsd": "^0.30.0"
},
"tsd": {
"directory": "./test/types"
}
},
"precommit": [
"test"
]
}

0 comments on commit 2cdc122

Please sign in to comment.