emacs-orgmode
[Top][All Lists]
Advanced

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

Re: New source block results option for attaching file to node


From: Timothy
Subject: Re: New source block results option for attaching file to node
Date: Tue, 31 Aug 2021 19:15:40 +0800
User-agent: mu4e 1.6.5; emacs 28.0.50

Hi Ryan,

Thanks for submitting your first patch to Org! 🎉

Regarding the purpose of the patch, I take it this is useful with source blocks that return a file patch and should be added as an attachment? Would you mind expanding on this use case? Oh, and org-attach-dir doesn’t show up a function or variable for me, I take it you meant something else?

Lastly, with your commit message, you’ve clearly looked at the requested commit message format 😊. There’s the odd typo (“theh”) and the line length is too long. It’s good if you hard line-wrap to 72 characters (if you use Magit, this should happen automatically FYI). Other than that it looks good to me.

All the best,
Timothy

From: Ryan Scott
Subject: New source block results option for attaching file to node
To: emacs-orgmode@gnu.org
Date: Thu, 26 Aug 2021 16:48:50 +0800

An additional option for use with ":results file" that moves the returned path to the node attachment directory (as returned by org-attach-dir), creating it if necessary.

First time submitting a patch. Any feedback is appreciated.

-ryan


reply via email to

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