-
Notifications
You must be signed in to change notification settings - Fork 437
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
Reduce wai-website build time #1077
Comments
One important limit to keep in mind when we want to address that, from jekyll-include-cache README:
This sometimes explains why |
About this part, we may reconsider how suggested translation priorities for each language are featured. This could be an alternative path to refactoring the existing translation sitemaps. |
Tipuesearch also plays a role in the repository size: see #771 (comment) |
Jekyll currently takes over 2 minutes to run in this repo. This impacts not only PR and deployment builds, but also individual builds when running in local development, which can slow down local testing considerably.
Running with
--profile
, the following files show up at the top of the list:include_cached
in more places where we aren't yet.act_rule.html
layout (19 seconds for 156 runs) - I don't see anyinclude
s in it. Maybe there are a few pieces we can meaningfully break into cached includes?tipuesearch_content.js
can be converted to a plugin and if that would speed up its generation? (I realize this is entirely a third-party file, but a single run of it accounts for 13 seconds in itself...)Hacky workaround
In the interim, I created a branch on my fork of wai-website-theme which can reduce the build time by almost a full minute if you do not need the translation sitemaps or tipue search. It can be used by updating the
remote_theme
line in_config.yml
(don't commit this change):Even then, it still arguably takes an unreasonably long time. Ideally, the build should take well under one minute (or at least incremental builds should), but I'm not sure that much of an improvement is feasible with the amount of content and the current architecture.
The text was updated successfully, but these errors were encountered: