Skip to content

Revert "add sf vm ssh (#107)"#110

Merged
coffinsfcompute merged 1 commit into
mainfrom
coffin/revert_ssh
Apr 11, 2025
Merged

Revert "add sf vm ssh (#107)"#110
coffinsfcompute merged 1 commit into
mainfrom
coffin/revert_ssh

Conversation

@coffinsfcompute
Copy link
Copy Markdown
Contributor

This reverts commit 85577a2.

This reverts commit 85577a2.
@semanticdiff-com
Copy link
Copy Markdown

semanticdiff-com Bot commented Apr 11, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  deno.lock Unsupported file format
  package.json  0% smaller
  src/helpers/urls.ts  0% smaller
  src/lib/ssh.ts  0% smaller
  src/lib/vm.ts  0% smaller

@coffinsfcompute coffinsfcompute merged commit 84f2f60 into main Apr 11, 2025
1 check passed
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR reverts the addition of the sf vm ssh command, removing SSH functionality from the CLI tool.

  • Removed vms_ssh_get endpoint from apiPaths in src/helpers/urls.ts
  • Removed the entire src/lib/ssh.ts file which contained the SSH command implementation
  • Removed registerSsh import and function call from src/lib/vm.ts
  • Removed SSH-related dependencies @alphahydrae/exec and shescape from package.json
  • Reordered VM-related endpoints in the API paths configuration

4 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@coffinsfcompute coffinsfcompute deleted the coffin/revert_ssh branch April 11, 2025 05:45
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.

1 participant