Skip to content

Commit

Permalink
feat: adding support for node 18 (#495)
Browse files Browse the repository at this point in the history
* feat: adding support for node 18

* test: fixing a broken test

* chore: update comment

Comment now reflects what the `engines.node` field looks like

Co-authored-by: Kanad Gupta <kgupta@umn.edu>
  • Loading branch information
erunion and kanadgupta authored Apr 20, 2022
1 parent d331343 commit 774407d
Show file tree
Hide file tree
Showing 5 changed files with 265 additions and 300 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [14.x, 16.x, 17.x]
node-version:
- 14
- 16
- 18

steps:
- uses: actions/checkout@v3
Expand Down
2 changes: 1 addition & 1 deletion __tests__/lib/isSupportedNodeVersion.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ describe('#isSupportedNodeVersion()', () => {
it('should return true for a supported version of node', () => {
expect(isSupportedNodeVersion('14.5.1')).toBe(true);
expect(isSupportedNodeVersion('16.0.0')).toBe(true);
expect(isSupportedNodeVersion('18.0.0')).toBe(true);
});

it('should return false for an unsupported version of node', () => {
expect(isSupportedNodeVersion('10.0.0')).toBe(false);
expect(isSupportedNodeVersion('12.0.0')).toBe(false);
expect(isSupportedNodeVersion('18.0.0')).toBe(false);
});
});
Loading

0 comments on commit 774407d

Please sign in to comment.