Skip to content
This repository has been archived by the owner on May 11, 2022. It is now read-only.

Conversation

bfitzpat
Copy link
Collaborator

@bfitzpat bfitzpat commented Dec 17, 2020

Signed-off-by: Brian Fitzpatrick bfitzpat@redhat.com

Meant to address cannot read property fsPath of undefined issue when reading from cache

image

Apparently the global or workspace uri locations are not defined in theia - Artem opened eclipse-theia/theia#8883

In this case, we are simply trying both the global and workspace locations

Signed-off-by: Brian Fitzpatrick <bfitzpat@redhat.com>
@bfitzpat bfitzpat self-assigned this Dec 17, 2020
@bfitzpat bfitzpat added the work in progress A draft PR or work in progress label Dec 17, 2020
@bfitzpat
Copy link
Collaborator Author

This was meant to address the issue on Che/Theia returning an error, but turned into a way to look at where we should cache these files. I suspect we should maybe shift to just use the workspace location rather than the global one to simplify things for Che and standard VS Code, but I'm not sure.

@bfitzpat
Copy link
Collaborator Author

Closing this as no longer needed after #365

@bfitzpat bfitzpat closed this Jan 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
work in progress A draft PR or work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant