infra: update copy druid script #519
Merged
+7
−3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Docusaurus 3 uses MDXv3, which is much stricter.
The longstanding way we've done variables is to find/replace
{{DRUIDVERSION}}
in the Markdown files. It worked in 2 b/c 2 was less strict. 3 is stricter and picks them up as Javascript variables that are undefined.Instead of making people learn a new way to do variables, this updates the copy script so that an escaped version of
\{\{DRUIDVERSION}}
(and variations) also get find/replacedWe would have to do a find/replace anyway unless we set up the variables to match each other in
druid-website-src
anddruid
To verify this, check out the Markdown cleanup branch in apache/druid#17740 and run the copy script in
scripts
.Then grep for
DRUIDVERSION
in the markdown files and maybe whatever version you set, eg 32.0.0