Make each command have a specific req'd cwms-python target

オープン
#151 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
機能追加
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
python
領域
cli

調査の方向性

まず cwmscli/requirements.py と cwmscli/commands/commands_cwms.py のコマンド定義を読み、次に pyproject.toml のグローバル依存関係を調べます。現在、依存関係がすべてのコマンドにどのように適用されているか、またコマンド固有のターゲットをどのように表現できるかを確認します。完了条件は、コマンドが必要な cwms-python のバージョンを明示的に宣言し、グローバル最小バージョンが CLI にとって引き続き適切であることです。

索引モデルが issue の本文から書いたものです。

説明

enhancement
CLI Script

other

Feature Proposal

I.e.

This

https://github.com/HydrologicEngineeringCenter/cwms-cli/blob/1bedb0fa92e62d69b31199888d5824949b08460a/cwmscli/requirements.py#L5-L11

just gets used on everything via

https://github.com/HydrologicEngineeringCenter/cwms-cli/blob/1bedb0fa92e62d69b31199888d5824949b08460a/cwmscli/commands/commands_cwms.py#L28

Thoughts on me making an issue to remove the global and make those explicit?

i.e.

​@requries(reqs.cwms.set("version", "1.0.3") This would not work

This will make it so a sub command like users can be set to 1.0.3 but the people also wanting to just have it work for another command can have an older version if they don't need to update!

REDACTED relevant log output

Related Context or Links

There's a global dep in the pytoml here

https://github.com/HydrologicEngineeringCenter/cwms-cli/blob/1bedb0fa92e62d69b31199888d5824949b08460a/pyproject.toml#L21

That could also be at a lower minimum required for all of cwms-cli. Although cwms-cli itself does not need cwms-python until you drill down.

主要言語
Python
スター
3
フォーク
5
PR マージ指標
30日以内にマージされた PR はありません

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

HydrologicEngineeringCenter/cwms-cli のほかの issue

HydrologicEngineeringCenter/cwms-cli の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。