-
Notifications
You must be signed in to change notification settings - Fork 165
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
Docusaurus Build/Deploy Broken #246
Comments
I just merged #247, which hopefully resolves problem two |
I attempted to reproduce and investigate the problem in an attempt to resolve this. As it turns out, the large mdx file seemed to be correct and the build failure was not reproduced. Perhaps thanks to the fix in #247, the build succeeded fine in the local environment and on render.com. However, it is slow! It takes about 10 minutes. I looked into the mdx files generated. Indeed, 5 of them, including I was concerned that I have tried several times to build & deploy using render.com so far and never reproduced the build failures you described in the third one. In the end, it seems that the current bsky lexicon inevitably slows down the build because of the large --- a/atproto-openapi-types/main.ts
+++ b/atproto-openapi-types/main.ts
@@ -70,6 +70,10 @@ for await (const entry of entries) {
components.schemas![identifier] = convertArray(id, name, def);
break;
case "object":
+ if (id == "app.bsky.feed.defs" && name == "postView") {
+ components.schemas![identifier] = {};
+ break;
+ }
components.schemas![identifier] = convertObject(id, name, def);
break;
case "procedure": { |
@sugyan thanks! Giving those tips a try. Also got a tip from render.com support to lock the Node version, specifically to avoid version 23.2: nodejs/node#55826 |
Ok! Progress on this...
Horray! I'm going to call this 50/50 render.com support and @sugyan. We already (happily) pay for render, so i'll offer the full bounty (https://bsky.app/profile/bnewbold.net/post/3lbhsy3uncs2r) to sugyan. |
Building this site on
main
branch is very slow and not working at all in our deploy environment (render.com).There are at least two problems, both related to the auto-generated API reference docs:
tools.ozone.setting.removeOptions
)I have a branch #236 which fixes the second issue (simple patch), and desperately tries to work around the first issue by removing many of the MDX files (just rips API docs out of the site). This results in a working and faster build locally, but still fails to deploy due to the third issue, which I can't reproduce locally.
I suspect that the first issue might be caused by recursive/nested Lexicon references getting pulled in to generated docs, and an acceptable fix/mitigation might be to truncate the depth of nesting.
The second is probably fixed by just cherry-picking my commit.
The third I have no idea. Might need to create a temporary render.com account to trigger/debug.
As some high-level context, we have a plan to move generated API reference docs out of this repo and website and in to a generic Lexicon documentation site/service for all Lexicions in the ecosystem. So don't want to invest too much time in to that aspect. But will probably continue to use docusaurus for the dev blog, tutorials, and other docs.
Clarifications
I merged #247 which hopefully fixed problem two.
This repo / build process has multiple stages:
The text was updated successfully, but these errors were encountered: