Skip to content

Commit

Permalink
Update for Fastify v5 (#36)
Browse files Browse the repository at this point in the history
* update

* nup

* use secure-json-parse workflow to test on as many versions as possible

* add typescript

* revert change tap

* add coverage

* remove browsers

* start from 16

* use c8@7
  • Loading branch information
gurgunday authored Apr 17, 2024
1 parent 6c60c16 commit 8dc74c9
Show file tree
Hide file tree
Showing 3 changed files with 118 additions and 16 deletions.
122 changes: 110 additions & 12 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,21 +3,119 @@ name: CI
on:
push:
branches:
- main
- master
- next
- 'v*'
- main
- master
- next
- "v*"
paths-ignore:
- 'docs/**'
- '*.md'
- "docs/**"
- "*.md"
pull_request:
paths-ignore:
- 'docs/**'
- '*.md'
- "docs/**"
- "*.md"

jobs:
dependency-review:
name: Dependency Review
if: github.event_name == 'pull_request'
runs-on: ubuntu-latest
permissions:
contents: read
steps:
- name: Check out repo
uses: actions/checkout@v4
with:
persist-credentials: false

- name: Dependency review
uses: actions/dependency-review-action@v4

lint:
name: Lint Code
runs-on: ubuntu-latest
permissions:
contents: read
steps:
- name: Check out 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 i --ignore-scripts

- name: Lint code
run: npm run lint

test:
uses: fastify/workflows/.github/workflows/plugins-ci.yml@v3
with:
license-check: true
lint: true
name: Test
runs-on: ubuntu-latest
permissions:
contents: read
strategy:
matrix:
node-version: [10, 12, 14, 16, 18, 20]
steps:
- name: Check out repo
uses: actions/checkout@v4
with:
persist-credentials: false

- name: Setup Node ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

- name: Upgrade npm
if: ${{ success() && matrix.node-version == '6' }}
run: npm i [email protected] -g

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

- name: Run tests
run: npm run test:unit

typescript:
name: Test TypeScript
runs-on: ubuntu-latest
permissions:
contents: read
steps:
- name: Check out 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 i --ignore-scripts

- name: tsd
run: npm run test:typescript

automerge:
name: Automerge Dependabot PRs
if: >
github.event_name == 'pull_request' &&
github.event.pull_request.user.login == 'dependabot[bot]'
needs: [lint, test, typescript]
permissions:
pull-requests: write
contents: write
runs-on: ubuntu-latest
steps:
- uses: fastify/github-action-merge-dependabot@v3
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
target: major
2 changes: 2 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,10 @@ const types = parse.types

function safeRegex (re, opts) {
if (!opts) opts = {}
/* c8 ignore next */
const replimit = opts.limit === undefined ? 25 : opts.limit

/* c8 ignore next 2 */
if (isRegExp(re)) re = re.source
else if (typeof re !== 'string') re = String(re)

Expand Down
10 changes: 6 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,17 @@
"ret": "~0.5.0"
},
"devDependencies": {
"standard": "^17.0.0",
"tape": "^5.0.0",
"@fastify/pre-commit": "^2.1.0",
"c8": "^7.14.0",
"standard": "^17.1.0",
"tape": "^5.7.5",
"tsd": "^0.31.0"
},
"scripts": {
"lint": "standard",
"test": "npm run test:unit",
"test": "npm run test:unit && npm run test:typescript",
"test:typescript": "tsd",
"test:unit": "tape test/*.js"
"test:unit": "c8 tape test/*.js"
},
"repository": {
"type": "git",
Expand Down

0 comments on commit 8dc74c9

Please sign in to comment.