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

Fix bad walk #6

Merged
merged 2 commits into from
Jan 24, 2024
Merged

Fix bad walk #6

merged 2 commits into from
Jan 24, 2024

Conversation

j2kun
Copy link
Contributor

@j2kun j2kun commented Jan 24, 2024

Fixes #4

Fixes gohugoio#4

The main approach is to split the walk into two walks:

1. Insert the new nodes that would replace the original node, with care
to avoid infinite loops, and mark the replaced nodes for deletion
2. Find the marked nodes and delete them.
passthrough/passthrough.go Outdated Show resolved Hide resolved
@jmooring jmooring merged commit 4a3408c into gohugoio:main Jan 24, 2024
6 checks passed
@jmooring
Copy link
Member

@bep @j2kun

I'm comfortable with tagging this passthrough/v0.1.0 but your call.

@j2kun
Copy link
Contributor Author

j2kun commented Jan 24, 2024

Fine by me.

@jmooring
Copy link
Member

jmooring commented Jan 24, 2024

@j2kun Done. Please update go.mod and run go mod tidy on the PR in the Hugo repo.

@jmooring
Copy link
Member

@j2kun I'll take a swing at Hugo documentation over the next couple of days, probably adding "math" page (title TBD) to the "Content management" section (https://gohugo.io/content-management). Need to cover config, JS inclusion methods, MathJax and KaTeX tradeoffs, examples, etc. After release, we can enable this on the docs site and include "live" examples.

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.

passthrough: Inconsistent parsing
3 participants