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

Introduce new subcmd edit #278

Merged
merged 7 commits into from
Apr 26, 2019
Merged

Introduce new subcmd edit #278

merged 7 commits into from
Apr 26, 2019

Conversation

hupfdule
Copy link
Contributor

Allows editing the plugconf file(s) of the given repositories.

Closes #230

There are still a few FIXMEs in the code.
The most important one being:

  • Which error codes to use?
    Currently 15 and 20 are used, but the existing error codes are not
    documented. It is unclear whether to reuse an existing one or which to
    use for which purpose.

The remaining ones are more about clean code and a different option

tyru and others added 3 commits October 27, 2018 07:41
Allows editing the plugconf file(s) of the given repositories.

Closes vim-volt#230

There are still a few FIXMEs in the code.
The most important one being:

- Which error codes to use?
  Currently 15 and 20 are used, but the existing error codes are not
  documented. It is unclear whether to reuse an existing one or which to
  use for which purpose.

The remaining ones are more about clean code and a different option
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
subcmd/edit.go Outdated Show resolved Hide resolved
@tyru
Copy link
Member

tyru commented Apr 18, 2019

Addition to above, could you add an simple example of edit.editor config value to README.md ?
https://github.com/vim-volt/volt#config

@tyru
Copy link
Member

tyru commented Apr 18, 2019

And please change target branch to vim-volt:devel .

@hupfdule hupfdule changed the base branch from master to devel April 20, 2019 14:33
@hupfdule hupfdule force-pushed the subcmd_edit branch 2 times, most recently from 15cce6d to 3621dfe Compare April 20, 2019 15:07
Copy link
Member

@tyru tyru left a comment

Choose a reason for hiding this comment

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

Thanks. Also modifying bash script is a good catch.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@tyru tyru merged commit 3afe097 into vim-volt:devel Apr 26, 2019
@tyru
Copy link
Member

tyru commented Apr 26, 2019

Thanks!
Will release the new version!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants