Replace and rewrite "assert" statements in non-test code
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 55/100
Direzione di ricerca
Inizia dalla ricerca di codice GitHub collegata per individuare le istruzioni assert Python non appartenenti ai test nei repository di aboutcode-org. Per ogni assert nel codice principale, stabilisci quale eccezione esplicita a runtime e quale messaggio dovrebbero sostituirlo, quindi esegui i test pertinenti del progetto. Il lavoro è completo quando gli assert non appartenenti ai test sono stati rimossi o giustificati e il comportamento continua a essere coperto dai test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
See https://www.reddit.com/r/learnpython/comments/zd217y/is_using_assert_considered_good_practice/
assert statements should only be used for debugging purposes. They are removed when not running in debug mode (i.e. when invoking the Python command with the -O or -OO options).
Instead use a raise statement to throw a proper, informative runtime exception, if you need to. That would definitely be best practice when there'd otherwise be no sensible return value for the function.
We are using a few asserts in our main code. See https://github.com/search?q=org%3Aaboutcode-org+NOT+path%3A*test*++"assert"++language%3APython&type=code
- Lingua principale
- Batchfile
- Stelle
- 310
- Fork
- 255
- Merge medio
- 10h 5m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 aboutcode-org/aboutcode
-
Cross-reference OpenChain CRA Compliance ChecklistForse di nuovo libera @PaulitePavitra l’ha presa 21 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
aboutcode-org/aboutcode#309 · 6 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
aboutcode-org/aboutcode#314 · 4 commenti ·
-
Project: 5C: Continuous Code Compliance Control CenterForse di nuovo libera @nt5504289-oss l’ha presa 33 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
aboutcode-org/aboutcode#291 · 3 commenti ·
-
Remove Projects section in READMEForse già presa @mjherzog l’ha presa 245 giorni fa. Apertadocumentation
aboutcode-org/aboutcode#277 · 6 commenti · 1 assegnatario ·
-
RFC: Create AI policyForse di nuovo libera @tdruez l’ha presa 252 giorni fa e non c’è nessuna pull request aperta. Apertapolicy
aboutcode-org/aboutcode#265 · 10 commenti · 1 assegnatario ·
Tutte le issue di aboutcode-org/aboutcode
Issue simili
-
.NET ep:WebGPU platform:web
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
microsoft/onnxruntime#33755 ·
I maintainer di solito rispondono entro 2 giorni
-
changelog investigate
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
ramnes/notion-sdk-py#433 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
intake/intake-esm#789 ·
-
area: core priority: low type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
RicoSuter/Namotion.Interceptor#863 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
MemMachine/MemMachine#1877 ·
I maintainer di solito rispondono entro 3 giorni