DOI-USGS / DOI-USGS/dataretrieval-python

enums for site type, county, etc.

Ouverte
#98 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.