Skip to content

Set default editor#14

Merged
tsub merged 1 commit intotsub:masterfrom
koooge:editor_default
Aug 22, 2018
Merged

Set default editor#14
tsub merged 1 commit intotsub:masterfrom
koooge:editor_default

Conversation

@koooge
Copy link
Copy Markdown
Contributor

@koooge koooge commented Aug 22, 2018

s3-edit tries to exec the target file on s3 when EDITOR is not set. It's not good 🤕

/tmp/README.md: line 1: qwerty: command not found
/tmp/README.md: line 2: qawetp: command not found

so, set vi as a default EDITOR.

@tsub
Copy link
Copy Markdown
Owner

tsub commented Aug 22, 2018

I did not notice it! Thank you! 😄

@tsub tsub merged commit 45258f7 into tsub:master Aug 22, 2018
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.

2 participants