avajs/ava

Support `.ava` cacheDir for better compatibility with Yarn PnP

Offen

#3.226 geöffnet am 28.07.2023

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.408 Forks)batch import
enhancementhelp wantedscope:internals

Repository-Metriken

Stars
 (20.600 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Hello,

We're using Yarn 3 with PnP in our mono-repo, and find that ava is creating unwanted node_modules directories in our workspace (with node_modules/.cache/ava/failing-tests.json) This is a result of https://github.com/avajs/ava/blob/f047694d86a8ee7af2e998baf28812770e9ea71e/lib/cli.js#L256

Sometimes the existence of node_modules confuses the pnp patched executables (eg. eslint), so the mere existence of node_modules can sometimes cause issues with things working reliable.

It would be great if you supported some manner of specifying the cache directory or simply defaulted to projectDir/.ava and recommended adding .ava to the .gitignore.

Thanks!

Contributor Guide

Research-Richtung
Überprüfe die Cache Verzeichniseinrichtung in lib/cli.js in Zeile 256. Verstehe, wie der Pfad bestimmt wird, und erwäge, eine CLI Option oder eine Umgebungsvariable hinzuzufügen, um ein alternatives Cache Verzeichnis anzugeben, das standardmäßig `.ava` im Projektstammverzeichnis verwendet. Stelle sicher, dass die Änderung das bestehende Verhalten nicht beeinträchtigt, und aktualisiere entsprechend die Dokumentation und die `.gitignore` Empfehlungen.
Tech Stack
nodejsjavascript
Domain
testingdevops
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Node.jsYarn PnP
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65