Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable

[Ihor Radchenko]

Tom Gillespie <tgbugs@gmail.com> writes:

> Got it. Here's the updated patch (I think I did it correctly?).

Thanks!
You also need a NEWS entry here.

> * lisp/ob-core.el (org-confirm-babel-evaluate-cell): Added to control
> execution of cells separate from execution of src blocks, it works in
> exactly the same way as org-confirm-babel-evaluate.

Please quote `symbols' in the commit message and separate sentences with
double space.

> +;; don't allow this variable to be changed through file settings
> +(put 'org-confirm-babel-evaluate-cell 'safe-local-variable (lambda (x) (eq x 
> t)))

You can simply use :safe keyword for `defcustom'.

I am also wondering if we should include this into bugfix.
This is a new feature, but it appears to be important enough to be
merged into built-in Org.
Bastien, Kyle, any thoughts?

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>