DOI-USGS / DOI-USGS/dataretrieval-python
enums for site type, county, etc.
- Langage dominant
- Python
- Étoiles
- 265
- Forks
- 63
- Merge moyen
- 1 j 20 h
- PR mergées (30 j)
- 19
Description
How do we feel about using [`enum`](https://docs.python.org/3.8/library/enum.html) for [site type](https://github.com/DOI-USGS/dataretrieval-python/blob/87351a13b7c7c1cf035e9a359addf289d14f5e50/dataretrieval/nwis.py#L609-L613), [county code](https://github.com/DOI-USGS/dataretrieval-python/blob/87351a13b7c7c1cf035e9a359addf289d14f5e50/dataretrieval/nwis.py#L583-L586C15), etc., instead of `str`?
It would fit nicely with:
- #97
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par lire dataretrieval/nwis.py au niveau des définitions des types de site et des codes de comté sur le site lié, puis examinez la proposition associée dans #97. Déterminez quelles valeurs devraient devenir des enums et comment cette modification s'intègre à l'API existante ; le travail sera considéré comme terminé lorsque la portée et la conception des enums seront convenues, car cet issue demande actuellement une discussion plutôt que de spécifier une implémentation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- api
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100