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

Enable fast mode #157

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Enable fast mode #157

merged 1 commit into from
Jun 14, 2024

Conversation

jcheng5
Copy link
Contributor

@jcheng5 jcheng5 commented Jun 13, 2024

This has two effects: making the build faster, as quarto render currently uses a slow Lua JSON parser to deserialize objects.json on every page render; and generating an objects.txt file, which can be used by sphinx so that downstream projects can interlink with us.

This has two effects: making the build faster, as quarto render currently
uses a slow Lua JSON parser to deserialize objects.json on every page
render; and generating an objects.txt file, which can be used by sphinx
so that downstream projects can interlink with us.
Copy link

@machow machow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM--I didn't look closely, but the fast: flag should be stable, and will likely become the default soonish

@jcheng5
Copy link
Contributor Author

jcheng5 commented Jun 14, 2024

The site successfully built with API docs in place, that's good enough for me!

@jcheng5 jcheng5 merged commit c6cba08 into main Jun 14, 2024
1 check passed
@jcheng5 jcheng5 deleted the fast-mode branch June 14, 2024 16:27
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.

None yet

2 participants