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.

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

new feature: introduce command oli edit #5593

Open
1 task done
TennyZhuang opened this issue Feb 2, 2025 · 3 comments
Open
1 task done

new feature: introduce command oli edit #5593

TennyZhuang opened this issue Feb 2, 2025 · 3 comments
Labels
enhancement New feature or request

Comments

@TennyZhuang
Copy link
Contributor

TennyZhuang commented Feb 2, 2025

Feature Description

oli edit [path]
  1. Load a file from the opendal path if exists to a local temp file.
  2. Using $EDITOR or vim to edit the temp file.
  3. When the editor quit, upload the temp file back to the path if changed.

Problem and Solution

How to handle the write error when put failed?

Solution 1: output the temp path in the console, and leave it to the user.
Solution 2: Open the temp file using $EDITOR again.

Additional Context

No response

Are you willing to contribute to the development of this feature?

  • Yes, I am willing to contribute to the development of this feature.
@TennyZhuang TennyZhuang added the enhancement New feature or request label Feb 2, 2025
@Xuanwo
Copy link
Member

Xuanwo commented Feb 2, 2025

Sounds like ocat s3://path/to/file | vipe | otee s3://path/to/file?

@TennyZhuang
Copy link
Contributor Author

Sounds like ocat s3://path/to/file | vipe | otee s3://path/to/file?

vipe is a very useful tool, but it seems unable to unify the cases where the key has not been created and where the key already exists.

@Xuanwo
Copy link
Member

Xuanwo commented Feb 2, 2025

Makes sense. There is such a tool called s3edit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants