emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] org-attach-commit optional


From: Nicolas Goaziou
Subject: Re: [O] org-attach-commit optional
Date: Thu, 11 Feb 2016 10:09:24 +0100

Hello,

John J Foerch <address@hidden> writes:

> I would like for org-attach's automatic committing of attachments to be
> easily disabled.  Something along the lines of the attached patch.
> Thoughts?

Thank you. A few stylistic comments follow.

> Subject: [PATCH] org-attach-attach: make use of git optional with variable
>  org-attach-commit

The commit message should be something along the lines:

  org-attach: Optionally make use of git

* lisp/org-attach.el (org-attach-commit): New variable.
(org-attach-attach): Use new variable.

You also need to add TINYCHANGE at the end of the commit message. On
this topic, please not that you are, with this patch, reaching the limit
of allowed changes without FSF assignment. You may want to consider
signing FSF papers.

> +(defcustom org-attach-commit t
> +  "If non-nil, attachments will be committed with git, if the org
> +file is in a git repository."
> +  :group 'org-attach
> +  :type 'boolean)

First line in docstring should be a sentence on its own, e.g.

    "Non-nil means attachements are to be committed with git.
  This can only happen if the Org file is in a git repository."

You should also add :version and :package-version keywords, e.g.,

  :version "25.1"
  :package-version '(Org . "9.0")


Regards,

-- 
Nicolas Goaziou



reply via email to

[Prev in Thread] Current Thread [Next in Thread]