Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: expose exclusion pattern for async-inject-variables as configuration var #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vidbina
Copy link

@vidbina vidbina commented Jun 29, 2022

Some variables matching the pattern org-babel-* caused some trouble because they had propertized texts for values and thus weren't serializable in a manner that didn't break reading attempts.

Since async-inject-variables allows for the definition of exclusion patterns, I simply used this to reduce the set of vars that actually get injected into the async process.

The default value for the exclusion pattern will drop variables that contain -overlay as part of their names since these typically contain overlays which have those propertized texts in my setup.

The previous variable ob-async-inject-variables has been kept as an alias for backwards compatibility.

@vidbina
Copy link
Author

vidbina commented Jun 30, 2022

I have some notes in my literate config on what the issue was if more details are helpful https://github.com/vidbina/dotfiles/tree/reorg-for-lit-prog/emacs#debug-invalid-read-read-syntax-error-during-variable-injection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant