Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

typo in pfs.ts #92860

Closed
videlalvaro opened this issue Mar 17, 2020 · 0 comments · Fixed by #92862
Closed

typo in pfs.ts #92860

videlalvaro opened this issue Mar 17, 2020 · 0 comments · Fixed by #92862
Assignees
@aeschli
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone
March 2020

Comments

@videlalvaro
Copy link
Contributor

Typo in pfs.ts here:

function toQueueKey(path: string): string {
	let queueKey = path;
	if (platform.isWindows || platform.isMacintosh) {
		queueKey = queueKey.toLowerCase(); // accomodate for case insensitive file systems
	}

	return queueKey;
}

accommodate
videlalvaro added a commit to videlalvaro/vscode that referenced this issue Mar 17, 2020
@videlalvaro
fixes microsoft#92860
98ae865
@videlalvaro videlalvaro mentioned this issue Mar 17, 2020
Merged
@aeschli aeschli added this to the March 2020 milestone Mar 17, 2020
@aeschli aeschli added the bug Issue identified by VS Code Team member as probable bug label Mar 17, 2020
@aeschli aeschli self-assigned this Mar 17, 2020
aeschli added a commit that referenced this issue Mar 17, 2020
@aeschli
Merge pull request #92862 from videlalvaro/issue_92860
60b847a 
fixes #92860
@dbaeumer dbaeumer added the verified Verification succeeded label Apr 2, 2020
@github-actions github-actions bot locked and limited conversation to collaborators May 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@aeschli aeschli

Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Milestone
March 2020
Development

Successfully merging a pull request may close this issue.

fixes #92860 videlalvaro/vscode
3 participants
@videlalvaro @dbaeumer @aeschli