`APPROVAL_TESTS_USE_REPORTER` should override the default reporter
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia tracciando ApprovalTests::Approvals::useAsFrontLoadedReporter e la gestione di APPROVAL_TESTS_USE_REPORTER. Conferma come vengono risolti il reporter predefinito e il reporter selezionato dall’ambiente; il lavoro è completato quando la variabile d’ambiente ha la precedenza senza richiedere modifiche al codice locale.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
My entire team prefers to use just the TextDiffReporter for day-to-day development, so we would like to do this in the code:
auto defaultReporter = ApprovalTests::Approvals::useAsFrontLoadedReporter(
std::make_shared<ApprovalTests::TextDiffReporter>());
However we would also like to use the APPROVAL_TESTS_USE_REPORTER environment variable to run with the AutoApproveIfMissingReporter when adding new tests, or AutoApproveReporter if we are making some bulk changes to our output. Or maybe in certain use cases or for certain developers they want to use a regular diff tool again. We would prefer not to have to do this via code because it ends up creating local changes that can be accidentally committed etc. And an #ifdef seems not so great if the environment variable is already provided.
But APPROVAL_TESTS_USE_REPORTER does not seem to override this default. It seems like it should to me.
- Lingua principale
- C++
- Stelle
- 340
- Fork
- 56
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di approvals/ApprovalTests.cpp
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
approvals/ApprovalTests.cpp#247 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
approvals/ApprovalTests.cpp#246 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
approvals/ApprovalTests.cpp#226 · 2 commenti ·
-
bug namers
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
approvals/ApprovalTests.cpp#225 · 5 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
approvals/ApprovalTests.cpp#217 · 4 commenti ·
Tutte le issue di approvals/ApprovalTests.cpp
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
hyprwm/aquamarine#426 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
amnezia-vpn/amnezia-client#3222 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
valkey-io/valkey-search#1465 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
KhronosGroup/Vulkan-Tutorial#524 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
microsoft/onnxruntime-genai#2633 ·
I maintainer di solito rispondono entro 1 giorno