-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
NodeExplorer: Implements Nodes list #106
Conversation
src/tsobj-file-system-provider.ts
Outdated
Logger.info(`hostname: ${hostname}`, 'tsobj-fsp'); | ||
Logger.info(`remotePath: ${resourcePath}`, 'tsobj-fsp'); | ||
|
||
const command = `ssh ${hostname} ls -lA "${resourcePath}"`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we just run ls -ap
here? it wouldn't require skipping lines or anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we can do ls -Ap
. A
will exclude the .
and ..
special entries.
src/tsobj-file-system-provider.ts
Outdated
const type = isDirectory ? vscode.FileType.Directory : vscode.FileType.File; | ||
files.push([name, type]); | ||
} | ||
resolve(files); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we sort the files so that directories show up first? This is how the regular file explorer sorts the display
vscode.TreeDragAndDropController<PeerBaseTreeItem>, | ||
vscode.FileDecorationProvider | ||
{ | ||
dropMimeTypes = ['text/uri-list']; // add 'application/vnd.code.tree.testViewDragAndDrop' when we have file explorer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this PR is missing this commit: 60dc7fa
At least the TypeScript part (not the Go part, since we're just shelling out to SSH)
This way we can have scp work between peers. We can also do it as a follow up PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, let's do that as a follow up.
Signed-off-by: Tyler Smalley <tyler@tailscale.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌🏼
This is the start of a Node Explorer, hidden behind a setting.