Lucas Oshiro <> writes:

> This series of patches fixes the flags --edit and --no-edit. Currently,
> --no-edit has no effect.
> These patches implement the following behaviour:
> - when --edit is provided, the editor will always be opened;
> - when --no-edit is provided, the editor will not be opened (if possible),
>   otherwise an error message will be displayed;
> - when neither --edit nor --no-edit are provided, the editor is opened only if
>   a message is not provided and there isn't a previous tag message.

Another thing that must be done is

 - when both --edit and --no-edit are given, the last one wins.

I do not know if you have implemented it in the three patches, though.

> In the future, the fix of these flags and the code factoring done in this
> patchset will be used on the implementation of a new flag --amend, as 
> discussed
> on the mail thread started on
> Lucas Oshiro (3):
>   tag: factor out tag reading from write_tag_body()
>   tag: factor out prepare tag template code
>   tag: add full support for --edit and --no-edit
>  builtin/tag.c  | 123 ++++++++++++++++++++++++++++++++++---------------
>  t/ |   4 +-
>  2 files changed, 88 insertions(+), 39 deletions(-)

Reply via email to