Skip to content

Conversation

Anemy
Copy link
Member

@Anemy Anemy commented May 9, 2020

@Anemy Anemy force-pushed the VSCODE-110/linux-keytar-tests branch from 0b78f4b to 6edd482 Compare May 14, 2020 09:48
@Anemy Anemy force-pushed the VSCODE-110/linux-keytar-tests branch from d03be8c to 89bf0cb Compare May 18, 2020 13:51
@Anemy Anemy marked this pull request as ready for review May 18, 2020 14:11
@Anemy Anemy requested a review from alenakhineika May 18, 2020 14:36
Copy link
Contributor

@alenakhineika alenakhineika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow super nice to have Linux tests back! LGTM!

src/extension.ts Outdated
import * as vscode from 'vscode';

import { ext } from './extensionConstants';
import { tryToCreateKeytar } from './utils/keytar';
Copy link
Contributor

@alenakhineika alenakhineika May 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just createKeytar? Try catch adds the same meaning later.

// in order to ensure we're not polluting the credential store
// and because its tough to get the credential store running on
// headless linux.
ext.keytarModule = new KeytarStub();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@Anemy Anemy merged commit 73d22ee into master May 19, 2020
@Anemy Anemy deleted the VSCODE-110/linux-keytar-tests branch May 19, 2020 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants