[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable
From: |
Ihor Radchenko |
Subject: |
Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable |
Date: |
Sun, 25 Dec 2022 11:06:41 +0000 |
Tom Gillespie <tgbugs@gmail.com> writes:
> From 4a78e1b5ea98dee569ff690037c661ab5c300194 Mon Sep 17 00:00:00 2001
> From: Tom Gillespie <tgbugs@gmail.com>
> Date: Sat, 10 Dec 2022 12:11:17 -0800
> Subject: [PATCH 1/2] ob-core: add org-confirm-babel-evaluate-cell custom
> variable
Bastien, may you please take a look at this discussion?
As pointed by this patch, the current behaviour of asking users about
code evaluation every single time will increase the odds to set
`org-confirm-babel-evaluate' to t and never come back.
We may not want to do it.
Instead, we may apply this patch, but using an opposite default value of
this new variable - do not confirm.
The end result after applying will then be the same as in the previous
release.
Or we may apply this patch + extend the confirmation dialogue to accept
"All" answer to confirm execution for the duration of `this-command'.
WDYT?
--
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>
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, (continued)
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Max Nikulin, 2022/12/15
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tim Cross, 2022/12/15
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/16
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tim Cross, 2022/12/16
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/18
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tim Cross, 2022/12/18
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tom Gillespie, 2022/12/19
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tom Gillespie, 2022/12/19
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/25
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/18
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable,
Ihor Radchenko <=
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Bastien Guerry, 2022/12/29
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Max Nikulin, 2022/12/29
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/29
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Bastien, 2022/12/30
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Max Nikulin, 2022/12/30
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tom Gillespie, 2022/12/30
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Ihor Radchenko, 2022/12/31
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Tom Gillespie, 2022/12/31
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Kyle Meyer, 2022/12/12
- Re: [PATCH] ob-core: add org-confirm-babel-evaluate-cell custom variable, Max Nikulin, 2022/12/13