From e936903d648caa40f4bbc7a6d0e28aeb59aedd43 Mon Sep 17 00:00:00 2001
From: Rebecca Stambler <rstambler@golang.org>
Date: Mon, 4 Nov 2019 21:56:57 -0500
Subject: [PATCH] Exclude prerelease tagged versions when fetching latest gopls
 version (#2885)

---
 src/goLanguageServer.ts | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/goLanguageServer.ts b/src/goLanguageServer.ts
index 6409077ec..25d3eed8e 100644
--- a/src/goLanguageServer.ts
+++ b/src/goLanguageServer.ts
@@ -507,13 +507,19 @@ async function latestGopls(tool: Tool): Promise<semver.SemVer> {
 	// Coerce the versions into SemVers so that they can be sorted correctly.
 	const versions = [];
 	for (const version of data.trim().split('\n')) {
-		versions.push(semver.coerce(version));
+		const parsed = semver.parse(version, {
+			includePrerelease: true,
+			loose: true,
+		});
+		versions.push(parsed);
 	}
 	if (versions.length === 0) {
 		return null;
 	}
 	versions.sort(semver.rcompare);
-	return versions[0];
+
+	// The first version in the sorted list without a prerelease tag.
+	return versions.find(version => !version.prerelease || !version.prerelease.length);
 }
 
 async function goplsVersion(goplsPath: string): Promise<string> {
@@ -594,7 +600,7 @@ async function goProxyRequest(tool: Tool, endpoint: string): Promise<any> {
 	return null;
 }
 
-function goProxy(): string[]  {
+function goProxy(): string[] {
 	const output: string = process.env['GOPROXY'];
 	if (!output || !output.trim()) {
 		return [];