Skip to content

Add caching of pi-gen artifacts#106

Open
usimd wants to merge 1 commit intomainfrom
add-caching
Open

Add caching of pi-gen artifacts#106
usimd wants to merge 1 commit intomainfrom
add-caching

Conversation

@usimd
Copy link
Copy Markdown
Owner

@usimd usimd commented Feb 23, 2024

Adds a new feature to allow caching of pi-gen build container. If a cache hit is found the container is loaded back into the Docker daemon and pi-gen is instructed to continue from the existing container.

@usimd usimd added the test Run integration test in PR label Feb 23, 2024
@usimd usimd force-pushed the add-caching branch 2 times, most recently from 573b143 to 0d449b0 Compare February 25, 2024 11:58
@github-actions github-actions Bot removed the test Run integration test in PR label Feb 25, 2024
@usimd usimd added the test Run integration test in PR label Feb 25, 2024
@github-actions github-actions Bot removed the test Run integration test in PR label Feb 25, 2024
@usimd usimd added the test Run integration test in PR label Feb 25, 2024
@usimd usimd force-pushed the add-caching branch 4 times, most recently from 6b908a2 to 4cf89c3 Compare February 26, 2024 10:18
@usimd usimd force-pushed the add-caching branch 2 times, most recently from 9abb29d to aa01c36 Compare May 15, 2024 18:37
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 7, 2024

Codecov Report

❌ Patch coverage is 99.18699% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 98.86%. Comparing base (4ff4525) to head (af5e183).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/cache.ts 98.64% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   98.45%   98.86%   +0.40%     
==========================================
  Files          13       16       +3     
  Lines         519      618      +99     
  Branches      118      135      +17     
==========================================
+ Hits          511      611     +100     
+ Misses          8        7       -1     
Flag Coverage Δ
unittests 98.86% <99.18%> (+0.40%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot removed the test Run integration test in PR label Nov 7, 2024
@usimd usimd added the test Run integration test in PR label Nov 7, 2024
@github-actions github-actions Bot removed the test Run integration test in PR label Nov 7, 2024
@usimd usimd added the test Run integration test in PR label Nov 8, 2024
@github-actions github-actions Bot removed the test Run integration test in PR label Nov 8, 2024
@usimd usimd added the test Run integration test in PR label Nov 8, 2024
@github-actions github-actions Bot removed the test Run integration test in PR label Nov 8, 2024
@github-actions github-actions Bot removed the test Run integration test in PR label Nov 8, 2024
@usimd usimd added the test Run integration test in PR label Nov 8, 2024
@usimd usimd force-pushed the add-caching branch 4 times, most recently from 2fac717 to fb235af Compare November 8, 2024 20:39
@github-actions github-actions Bot removed the test Run integration test in PR label Nov 8, 2024
@JanLahmann
Copy link
Copy Markdown

This pi-gen caching feature would be very helpful and could save a lot of time during development.
Is it possible to test it, already?

@usimd
Copy link
Copy Markdown
Owner Author

usimd commented Apr 21, 2025

Unfortunately not. I did not find enough time lately to continue working on this and the current approach is just a sketchy idea, no clue if that is actually going to improve anything.

If you would like to contribute, feel free to fork from the existing branch.

@usimd usimd force-pushed the add-caching branch 4 times, most recently from 9118f91 to 81e6aa5 Compare August 29, 2025 10:22
@usimd usimd added the test Run integration test in PR label Aug 29, 2025
@github-actions github-actions Bot removed the test Run integration test in PR label Aug 29, 2025
@usimd usimd added the test Run integration test in PR label Aug 30, 2025
@github-actions github-actions Bot removed the test Run integration test in PR label Aug 30, 2025
@usimd usimd added the test Run integration test in PR label Aug 31, 2025
@usimd usimd force-pushed the add-caching branch 4 times, most recently from 61cb9de to 0999485 Compare September 1, 2025 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Run integration test in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants