search

rkiggen / ob-powershell

Emacs Babel Powershell implementation
MIT License
13 stars 6 forks source link

Allow % in powershell source #5

Closed Shizcow closed 1 year ago

Shizcow commented 1 year ago

Elisp's message takes a format string as its first argument, meaning it will try to expand anything after a %. We want to simply pass through the literal text value, so (message "%s" full-body) should be used to avoid the additional expansion.

If this is kept as-is, then any time the user attempts to execute powershell code with a % present, emacs presents a user error Not enough arguments for format string.

Shizcow commented 1 year ago

IMO there shouldn't me (message full-body) at all -- it isn't standard (compared to other org-babel-execute:) and it pollutes `Messages*`.