Merge pull request #252 from jcreedcmu/jcreed/version-filter

jcreedcmu · web-flow · commit 018e9c0ae794 · 2020-02-28T12:03:27.000-05:00
Relax CLI version constraint to allow minor version increases
diff --git a/extensions/ql-vscode/src/distribution.ts b/extensions/ql-vscode/src/distribution.ts
@@ -39,9 +39,9 @@ const DEFAULT_DISTRIBUTION_REPOSITORY_NAME = "codeql-cli-binaries";
  * This applies to both extension-managed and CLI distributions.
  */
 export const DEFAULT_DISTRIBUTION_VERSION_CONSTRAINT: VersionConstraint = {
-  description: "2.0.*",
+  description: "2.*.*",
   isVersionCompatible: (v: Version) => {
-    return v.majorVersion === 2 && v.minorVersion === 0
+    return v.majorVersion === 2 && v.minorVersion >= 0
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -39,9 +39,9 @@ const DEFAULT_DISTRIBUTION_REPOSITORY_NAME = "codeql-cli-binaries";`
`39`	`39`	`* This applies to both extension-managed and CLI distributions.`
`40`	`40`	`*/`
`41`	`41`	`export const DEFAULT_DISTRIBUTION_VERSION_CONSTRAINT: VersionConstraint = {`
`42`		`- description: "2.0.*",`
	`42`	`+ description: "2..",`
`43`	`43`	`isVersionCompatible: (v: Version) => {`
`44`		`- return v.majorVersion === 2 && v.minorVersion === 0`
	`44`	`+ return v.majorVersion === 2 && v.minorVersion >= 0`
`45`	`45`	`}`
`46`	`46`	`}`
`47`	`47`