Skip to content

Commit

Permalink
build!: drop support for node 16 (#575)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: This module now requires node.js 18 and up. It includes a variety of related changes:
- Migrate from gts to xo for linting and code formatting
- Upgrade multiple deps to the latest version that were blocked on node 16
- Drop the codecov dependency and move to the github action
- Update the tsconfig to target Node16 for the module strategy

It's entirely possible this change didn't break anything for most users, but it's also possible it did.
  • Loading branch information
JustinBeckwith authored Oct 24, 2023
1 parent a530c40 commit 351a7c9
Show file tree
Hide file tree
Showing 27 changed files with 11,676 additions and 17,202 deletions.
4 changes: 0 additions & 4 deletions .eslintignore

This file was deleted.

3 changes: 0 additions & 3 deletions .eslintrc.json

This file was deleted.

12 changes: 8 additions & 4 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,14 @@ on:
name: ci
env:
FORCE_COLOR: 2
NODE: 18
NODE: 20
jobs:
test:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
node: [16, 18, 20]
node: [18, 20]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
Expand All @@ -23,8 +23,12 @@ jobs:
- run: node -v
- run: npm ci
- run: npm test
- run: npm run codecov
if: matrix.node == env.NODE
- run: npm run coverage
if: matrix.node == env.node
- uses: codecov/codecov-action@v3
if: matrix.node == env.node
with:
fail_ci_if_error: false
windows:
runs-on: windows-latest
steps:
Expand Down
1 change: 0 additions & 1 deletion .prettierignore

This file was deleted.

6 changes: 0 additions & 6 deletions .prettierrc.json

This file was deleted.

4 changes: 1 addition & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,8 @@
> A super simple site crawler and broken link checker.
[![npm version](https://img.shields.io/npm/v/linkinator)](https://www.npmjs.org/package/linkinator)
[![Build Status](https://img.shields.io/github/workflow/status/JustinBeckwith/linkinator/ci/main)](https://github.com/JustinBeckwith/linkinator/actions?query=branch%3Amain+workflow%3Aci)
[![codecov](https://img.shields.io/codecov/c/github/JustinBeckwith/linkinator/main)](https://app.codecov.io/gh/JustinBeckwith/linkinator)
[![Known Vulnerabilities](https://img.shields.io/snyk/vulnerabilities/github/JustinBeckwith/linkinator)](https://snyk.io/test/github/JustinBeckwith/linkinator)
[![Code Style: Google](https://img.shields.io/badge/code%20style-google-blueviolet)](https://github.com/google/gts)
[![XO code style](https://shields.io/badge/code_style-5ed9c7?logo=xo&labelColor=gray)](https://github.com/xojs/xo)
[![semantic-release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079)](https://github.com/semantic-release/semantic-release)


Expand Down
Loading

0 comments on commit 351a7c9

Please sign in to comment.