Skip to content

Commit

Permalink
fix(deps): -semver (#1893)
Browse files Browse the repository at this point in the history
chore: remove semver as dependency
  • Loading branch information
Uzlopak committed Nov 12, 2023
1 parent 4d031c1 commit 0251f68
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 12 deletions.
8 changes: 0 additions & 8 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions package.json
Expand Up @@ -64,7 +64,6 @@
"pino-http": "^5.3.0",
"pkg-conf": "^3.1.0",
"resolve": "^1.19.0",
"semver": "^7.3.4",
"update-dotenv": "^1.1.1"
},
"devDependencies": {
Expand All @@ -77,7 +76,6 @@
"@types/js-yaml": "^3.12.5",
"@types/node": "^18.18.1",
"@types/resolve": "^1.17.1",
"@types/semver": "^7.3.4",
"@types/supertest": "^2.0.10",
"body-parser": "^1.19.0",
"bottleneck": "^2.19.5",
Expand Down
4 changes: 2 additions & 2 deletions src/bin/probot.ts
@@ -1,11 +1,11 @@
import semver from "semver";
import program from "commander";
import { isSupportedNodeVersion } from "../helpers/is-supported-node-version";

require("dotenv").config();

const pkg = require("../../package");

if (!semver.satisfies(process.version, pkg.engines.node)) {
if (!isSupportedNodeVersion()) {
console.log(
`Node.js version ${pkg.engines.node} is required. You have ${process.version}.`,
);
Expand Down
3 changes: 3 additions & 0 deletions src/helpers/is-supported-node-version.ts
@@ -0,0 +1,3 @@
export function isSupportedNodeVersion(nodeVersion = process.versions.node) {
return Number(nodeVersion.split(".", 10)[0]) >= 18;
}
21 changes: 21 additions & 0 deletions test/bin/is-supported-node-version.test.ts
@@ -0,0 +1,21 @@
import { isSupportedNodeVersion } from "../../src/helpers/is-supported-node-version";
import { engines } from "../../package.json";

describe("isSupportedNodeVersion", () => {
it(`engines value is set to ">=18"`, () => {
expect(engines.node).toBe(">=18");
});

it("returns true if node is bigger or equal v18", () => {
expect(isSupportedNodeVersion("18.0.0")).toBe(true);
expect(isSupportedNodeVersion("19.0.0")).toBe(true);
expect(isSupportedNodeVersion("20.0.0")).toBe(true);
expect(isSupportedNodeVersion("21.0.0")).toBe(true);
});

it("returns false if node is smaller than v18", () => {
expect(isSupportedNodeVersion("17.0.0")).toBe(false);
expect(isSupportedNodeVersion("17.9.0")).toBe(false);
expect(isSupportedNodeVersion("17.9.9")).toBe(false);
});
});

0 comments on commit 0251f68

Please sign in to comment.