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

Fix gallery cache #4173

Merged
merged 2 commits into from
Mar 22, 2023
Merged

Fix gallery cache #4173

merged 2 commits into from
Mar 22, 2023

Conversation

akaszynski
Copy link
Member

I neglected to fix our gallery cache path in #4118 and it's causing our documentation builds to take forever.

@akaszynski akaszynski changed the title fix gallery cache Fix gallery cache Mar 22, 2023
@github-actions github-actions bot added the maintenance Low-impact maintenance activity label Mar 22, 2023
@akaszynski
Copy link
Member Author

Going to do an evil no review merge here, so review now or forever hold your peace.

@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

Merging #4173 (ad23188) into main (bee7cbc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4173   +/-   ##
=======================================
  Coverage   95.58%   95.58%           
=======================================
  Files          95       95           
  Lines       20356    20356           
=======================================
  Hits        19458    19458           
  Misses        898      898           

@tkoyama010
Copy link
Member

Going to do an evil no review merge here, so review now or forever hold your peace.

What is the original source?

@akaszynski
Copy link
Member Author

Going to do an evil no review merge here, so review now or forever hold your peace.

What is the original source?

It's said at some weddings:
https://www.brides.com/speak-now-or-forever-hold-your-peace-5100968

I'm mostly being silly here since I know there's the 24 hour rule, but this is one of those near-trivial fixes that I want to get through to reduce our build time from ~40 to ~20 minutes.

@akaszynski akaszynski merged commit bf44573 into main Mar 22, 2023
@akaszynski akaszynski deleted the ci/fix-gallery-cache branch March 22, 2023 03:01
@akaszynski akaszynski mentioned this pull request Apr 30, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants