DOI-USGS / DOI-USGS/dataretrieval-python
enums for site type, county, etc.
オープン
- 主要言語
- Python
- スター
- 265
- フォーク
- 63
- 平均マージ
- 1日 20時間
- マージ済み PR(30日)
- 19
説明
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
コントリビューションガイド
調査の方向性
まず、リンク先のサイトタイプと郡コードの定義がある dataretrieval/nwis.py を読み、次に #97 の関連する提案を確認してください。どの値を enum にすべきか、またその変更が既存の API にどう適合するかを判断します。この issue は現在、実装を指定するのではなく議論を求めているため、スコープと enum の設計について合意できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- api
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100