Merge pull request #1145 from Microsoft/avatar-detection-bugfix

StanleyGoldman · web-flow · commit 0bc5021c5951 · 2019-05-06T23:14:37.000+03:00
Adding missing else clause
diff --git a/src/github/githubRepository.ts b/src/github/githubRepository.ts
@@ -227,8 +227,9 @@ export class GitHubRepository implements IGitHubRepository, vscode.Disposable {
 			if (response && response.status === 200) {
 				this._repositoryReturnsAvatar = true;
 			}
-
-			this._repositoryReturnsAvatar = false;
+			else {
+				this._repositoryReturnsAvatar = false;
+			}
 		}
 
 		return this._repositoryReturnsAvatar;

Original file line number	Diff line number	Diff line change
`@@ -227,8 +227,9 @@ export class GitHubRepository implements IGitHubRepository, vscode.Disposable {`
`227`	`227`	`if (response && response.status === 200) {`
`228`	`228`	`this._repositoryReturnsAvatar = true;`
`229`	`229`	`}`
`230`		`-`
`231`		`- this._repositoryReturnsAvatar = false;`
	`230`	`+ else {`
	`231`	`+ this._repositoryReturnsAvatar = false;`
	`232`	`+ }`
`232`	`233`	`}`
`233`	`234`
`234`	`235`	`return this._repositoryReturnsAvatar;`