ParseVersionOrNone span test is dead code: PARSE_READ_ONLY_SPAN_SUPPORTED is never defined
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #994 di @seragAhmed — integrata
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Leggi Funcky.Test/Extensions/ParseExtensions/ParseExtensionsTest.Version.cs e confronta la relativa guardia delle funzionalità con la guardia GENERIC_PARSABLE in ParseExtensionsTest.Generic.cs; poi controlla le costanti delle funzionalità importate. Sostituisci la guardia non definita con READ_ONLY_SPAN_SUPPORTED, esegui il test pertinente e cerca altre costanti non definite nelle guardie del progetto di test. Il lavoro è terminato quando il test span viene compilato e superato sui target di test supportati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Funcky.Test/Extensions/ParseExtensions/ParseExtensionsTest.Version.cs guards ParseVersionIsTheSameAsTryParseForValidVersionsWithReadOnlySpan with
#if PARSE_READ_ONLY_SPAN_SUPPORTED
That constant is defined nowhere (not in FrameworkFeatureConstants.props, not in any csproj), so the test has never compiled or run since it was added in 5be57067 (Dec 2021). The ReadOnlySpan<char> overload of ParseVersionOrNone exists on every target with READ_ONLY_SPAN_SUPPORTED (netstandard2.1+), i.e. on all test TFMs except net4.8.
Fix: replace the guard with #if READ_ONLY_SPAN_SUPPORTED (the test project already imports the feature constants, GENERIC_PARSABLE is used the same way in ParseExtensionsTest.Generic.cs) and make sure the test passes. Worth a quick grep for other #if constants in the test projects that no props file defines.
- Lingua principale
- C#
- Stelle
- 55
- Fork
- 6
- Merge medio
- 1g 11m
- PR unite (30g)
- 80
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 polyadic/funcky
-
Ignore the .claude directoryForse già presa @FreeApophis l’ha presa 1 giorno fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
blocked
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
polyadic/funcky#868 · 5 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
I maintainer di solito rispondono entro 1 giorno
-
λ1012 code fix: Fix-All needs a second pass when three or more fixes add the same usingForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di polyadic/funcky
Issue simili
-
devtools documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
Azure.Identity bug Client
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Azure/azure-sdk-for-net#64293 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Live-Charts/LiveCharts2#2403 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
open-feature/dotnet-sdk#847 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100