controlled-versions: ignore git and file links (#5)

__tests__/rules/controlled-versions.spec.ts

@@ -76,6 +76,33 @@ tester.run("controlled-versions", rule, {
       filename: "package.json",
       options: [{ granularity: "minor", excludePatterns: ["foo*"] }],
     },
+    // ignores git links
+    {
+      code: `{
+        "name": "p1",
+        "dependencies": {
+          "foo1": "git://github.com/foo/foo1.git",
+          "foo2": "git+ssh://[email protected]:foo/foo2.git",
+          "foo3": "git+http://[email protected]/foo/foo3",
+          "foo4": "git+https://[email protected]/foo/foo4",
+          "foo5": "git+file:///path/to/file"
+        }
+      }`,
+      filename: "package.json",
+      options: [{ granularity: "minor" }],
+    },
+    // ignores file links
+    {
+      code: `{
+        "name": "p1",
+        "dependencies": {
+          "foo1": "file:../relative/path/to/file",
+          "foo1": "file:/full/path/to/file"
+        }
+      }`,
+      filename: "package.json",
+      options: [{ granularity: "minor" }],
+    },
   ],
   invalid: [
     // fixed without explicitly passing granularity

src/rules/controlled-versions.ts

@@ -7,6 +7,9 @@ import { Dependencies, DependencyGranularity } from "../types";
 import micromatch from "micromatch";
 import { toControlledSemver } from "../to-controlled-semver";
 
+const isGitDependency = (version: string): boolean => version.startsWith("git");
+const isFileDependency = (version: string): boolean => version.startsWith("file");
+
 const isFixedVersion = (version: string): boolean => {
   const cleanedSemver = cleanSemver(version);
 
@@ -101,7 +104,8 @@ const rule: Rule.RuleModule = {
             .pickBy(
               (_, dependency) =>
                 micromatch([dependency], excludePatterns).length === 0
-            )
+          )
+            .omitBy((version) => isGitDependency(version!) || isFileDependency(version!))
             .forEach((version, dependency) => {
               if (!version) {
                 return;