Skip to content

Introduce new subcmd edit - #278

Merged
tyru merged 7 commits into
vim-volt:develfrom
hupfdule:subcmd_edit
Apr 26, 2019
Merged

tyru merged 7 commits into
vim-volt:develfrom
hupfdule:subcmd_edit

Conversation

@hupfdule

Copy link
Copy Markdown
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
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
Comment thread subcmd/edit.go Outdated
@tyru

tyru commented Apr 18, 2019

Copy link
Copy Markdown
Member

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

tyru commented Apr 18, 2019

Copy link
Copy Markdown
Member

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

@tyru tyru left a comment

Copy link
Copy Markdown
Member

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.

Comment thread README.md Outdated
Comment thread README.md
@tyru
tyru merged commit 3afe097 into vim-volt:devel Apr 26, 2019
@tyru

tyru commented Apr 26, 2019

Copy link
Copy Markdown
Member

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