Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Silently pass on unsupported versions of Node.js #99

Merged
merged 1 commit into from
Dec 25, 2018

Conversation

watson
Copy link
Contributor

@watson watson commented Sep 12, 2018

While Node.js <6 haven't been officially supported by dependency-check since version 3, it did technically work until v3.2.1 was released as the debug dependency now uses the let keyword and therefore fails when being required.

This PR allows users to have depedency-check v3 as part of their CI build pipeline without being required to remove EoL versions of Node.js from their build matrix.

This escape hatch is similar to the one used in the standard linter.

This allows users to have depedency-check v3 as part of their CI build
pipeline without being required to remove EoL versions of Node.js from
their build matrix.
watson added a commit to watson/apm-agent-nodejs that referenced this pull request Sep 12, 2018
Version 3.2.1 was just released which no longer works on Node.js 4.
Version 3.x never officially supported Node.js 4, but did run fine until
now.

This commit and be reverted once the following PR is released:
dependency-check-team/dependency-check#99
watson added a commit to watson/apm-agent-nodejs that referenced this pull request Sep 12, 2018
Version 3.2.1 was just released which no longer works on Node.js 4.
Version 3.x never officially supported Node.js 4, but did run fine until
now.

This commit and be reverted once the following PR is released:
dependency-check-team/dependency-check#99
watson added a commit to elastic/apm-agent-nodejs that referenced this pull request Sep 12, 2018
Version 3.2.1 was just released which no longer works on Node.js 4.
Version 3.x never officially supported Node.js 4, but did run fine until
now.

This commit and be reverted once the following PR is released:
dependency-check-team/dependency-check#99
@voxpelli voxpelli merged commit 5aff1bc into dependency-check-team:master Dec 25, 2018
@voxpelli
Copy link
Member

Released as 3.3.0 @watson, sorry for the delay in merging – forgot about this PR. Thanks for sending PR:s 👍

@watson watson deleted the silent-pass branch December 28, 2018 13:18
@watson
Copy link
Contributor Author

watson commented Dec 28, 2018

No worries 😃

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants