-
Notifications
You must be signed in to change notification settings - Fork 56
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
[CI]: fix docs generation #359
Conversation
~/.elan/bin/lake exe cache get | ||
|
||
# Disable an error message due to a non-blocking bug. See Zulip | ||
MATHLIB_NO_CACHE_ON_UPDATE=1 ~/.elan/bin/lake update FLT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just echoing Ruben's comment on Zulip -- is there a reason we're not getting cache any more?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, because it's unnecessary since lake update
already does that.
# Clean up after ourselves | ||
rm -rf docbuild | ||
sudo chown -R runner docs | ||
cp -r docbuild/.lake/build/doc docs/docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So we are no longer cleaning up after ourselves?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We save docbuild/.lake/build/doc
to cache so we need it and we don't want to remove it.
No description provided.