Replace and rewrite "assert" statements in non-test code
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 55/100
Piste de recherche
Commencez par la recherche de code GitHub liée concernant les instructions assert Python qui ne se trouvent pas dans les tests dans les dépôts aboutcode-org. Pour chaque assert du code principal, déterminez quelle exception d’exécution explicite et quel message devraient le remplacer, puis exécutez les tests pertinents du projet. C’est terminé lorsque les asserts qui ne se trouvent pas dans les tests ont été supprimés ou justifiés et que le comportement reste couvert par des tests.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- Batchfile
- Étoiles
- 310
- Forks
- 255
- Merge moyen
- 10 h 5 min
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de aboutcode-org/aboutcode
-
Cross-reference OpenChain CRA Compliance ChecklistPeut-être à nouveau libre @PaulitePavitra l’a pris il y a 21 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
aboutcode-org/aboutcode#309 · 6 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
aboutcode-org/aboutcode#314 · 4 commentaires ·
-
Project: 5C: Continuous Code Compliance Control CenterPeut-être à nouveau libre @nt5504289-oss l’a pris il y a 33 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
aboutcode-org/aboutcode#291 · 3 commentaires ·
-
Remove Projects section in READMEPeut-être pris @mjherzog l’a pris il y a 245 jours. Ouvertedocumentation
aboutcode-org/aboutcode#277 · 6 commentaires · 1 personne assignée ·
-
RFC: Create AI policyPeut-être à nouveau libre @tdruez l’a pris il y a 252 jours, et aucune pull request n’est ouverte. Ouvertepolicy
aboutcode-org/aboutcode#265 · 10 commentaires · 1 personne assignée ·
Toutes les issues de aboutcode-org/aboutcode
Issues similaires
-
.NET ep:WebGPU platform:web
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
microsoft/onnxruntime#33755 ·
Les mainteneurs répondent en général sous 2 jours
-
changelog investigate
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
ramnes/notion-sdk-py#433 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
intake/intake-esm#789 ·
-
area: core priority: low type: bug
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
RicoSuter/Namotion.Interceptor#863 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
MemMachine/MemMachine#1877 ·
Les mainteneurs répondent en général sous 3 jours