matrix-org / matrix-org/setup-python-poetry

Make use of `restore-keys`

Offen
#5 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Keine Sprachdaten
Sterne
1
Forks
5
Ø Merge
2 T. 22 Std.
Gemergte PRs (30 T.)
1

Beschreibung

If we don't have a cache-hit for the poetry-managed venv, we'll end up recreating it from scratch. We could save a little bit of time by restoring from a different cache with a similar name.

We could probably make use of restore-keys here:

https://github.com/matrix-org/setup-python-poetry/blob/71e0c442d1dff598316bd217a8c7afe86a83958f/action.yml#L65-L70

Probably restore-keys: poetry-venv-${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}?

Would need to test it by making a small change to the lockfile and seeing if we modify a cached venv rather than recreating it from scratch. Or we could just make the change and YOLO it.

Either way, it's one for the wishlist.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Öffne action.yml in den Zeilen 65–70 und überprüfe die bestehende Cache-Konfiguration. Füge das vorgeschlagene Verhalten von restore-keys hinzu und teste es anschließend, indem du eine kleine Änderung an der Lockdatei vornimmst und prüfst, ob die gecachte virtuelle Umgebung wiederverwendet wird, anstatt sie von Grund auf neu zu erstellen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
github-actions, python
Bereich
ci-cd
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.