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

perf: Use non-lazy AST traversal for filters when possible #11052

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

tarleb
Copy link
Collaborator

@tarleb tarleb commented Oct 13, 2024

  • Add jog Lua module
  • Use jog instead of walk
  • Force use of pandoc walk for some filters
  • Add mechanism to check jog results

Welcome to the quarto GitHub repo!

We are always happy to hear feedback from our users.

To file a pull request, please follow these instructions carefully: https://yihui.org/issue/#bug-reports

If you're a collaborator from outside quarto-dev making changes larger than a typo, please make sure you have filed an individual or corporate contributor agreement. You can send the signed copy to jj@rstudio.com.

Also, please complete and keep the checklist below.

Description

Please describe your PR here.

```py
print("Hello Quarto!")
```

Checklist

I have (if applicable):

  • filed a contributor agreement.
  • referenced the GitHub issue this PR closes
  • updated the appropriate changelog

@tarleb tarleb marked this pull request as draft October 13, 2024 16:18
Setting the `QUARTO_JOG_CHECK` environment variable will run checks to
identify filters for which the results of `jog` differ from those of
pandoc's `walk`.
This should be reversed at some point.
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