perf(backend): cache .venv path correctly #10

Merged

mcataford merged 2 commits from perf/ci-caching into main

2023-08-12 15:59:03 +00:00

mcataford commented

2023-08-12 15:49:30 +00:00

(Migrated from github.com)

The path input of caching actions was not set properly to account for the frontend/ and backend/ namespacing of each app. The confusion might have stemmed from the working directory default that's on run blocks but that wouldn't apply to uses.

Prefixing cache retrieval paths with the appropriate top-level directory allows the post-action in Setup to cache dependencies correctly, and each of the steps to retrieve it as they should.

🎉 Builds are now superfast.

The `path` input of caching actions was not set properly to account for the `frontend/` and `backend/` namespacing of each app. The confusion might have stemmed from the working directory default that's on `run` blocks but that wouldn't apply to `uses`. Prefixing cache retrieval paths with the appropriate top-level directory allows the post-action in _Setup_ to cache dependencies correctly, and each of the steps to retrieve it as they should. :tada: Builds are now superfast.

This repo is archived. You cannot comment on pull requests.

No reviewers