Hi Bastien, Thanks for the feedback. I have attached the modified patch. All best, Leonard On 18 April 2014 12:14, Bastien wrote: > Hi Leonard, > > thanks for starting this! A few stylistic comments inline. > > Leonard Randall writes: > > > diff --git a/lisp/org-bibtex.el b/lisp/org-bibtex.el > > index ed645e5..848d0e4 100644 > > --- a/lisp/org-bibtex.el > > +++ b/lisp/org-bibtex.el > > @@ -270,20 +270,31 @@ with underscores, and characters that are not > permitted in org > > tags will be removed. > > > > If t, local tags in an org entry will be exported as a > > -comma-separated string of keywords when exported to bibtex. Tags > > -defined in `org-bibtex-tags' or `org-bibtex-no-export-tags' will > > -not be exported." > > +comma-separated string of keywords when exported to bibtex. If > > Make sure to end sentences with two spaces. > > > (defcustom org-bibtex-no-export-tags nil > > "List of tag(s) that should not be converted to keywords. > > -This variable is relevant only if `org-bibtex-export-tags-as-keywords' > is t." > > +This variable is relevant only if `org-bibtex-tags-are-keywords' is t." > > :group 'org-bibtex > > :version "24.1" > > :type '(repeat :tag "Tag" (string))) > > I fixed this docstring from the maint branch, you may need to pull > again and rewrite your changes. > > > +(defcustom org-bibtex-inherit-tags nil > > + "This variable controlls whether inherited tags are included > > The first line of the docstring should be a sentence. > > > +when converting org tags to bibtex keywords. It is relevant only > > +if `org-bibtex-tags-are-keywords' is t. Tag inheritence itself is > > +controlled by `org-use-tag-inheritence' and > > +`org-exclude-tags-from-inheritence'" > > There is a missing fullstop at the end, and missing double-space > between sentences. > > > + :group 'org-bibtex > > + :version "24.1" > > + :type 'boolean) > > Use > > :version "25.1" > :package-version '(Org . "8.3") > > here, so that users will know the option is new in Org 8.3 (the next > version that will be released from the master branc) and in Emacs 25.1 > (the next Emacs stable version that will contain 8.3.) > > Otherwise, the patch looks good. Please resubmit it from a fresh pull > with the modifications I suggested. > > Thanks in advance! > > -- > Bastien >