Remove dep on traitlet
まだ誰も着手していません。
評価
調査の方向性
まず、comm パッケージが traitlets.utils.importstring をインポートしている箇所を特定し、issue でリンクされている traitlets の importstring.py 実装と比較します。パッケージが明示している Python 3.6 のサポートを維持したまま依存関係を削除し、その後、既存のテストスイートを実行します。issue では特定のファイルやテストは指定されていません。
索引モデルが issue の本文から書いたものです。
説明
Hi!
as we discovered, the dependency on traitlet >= 5 means that actually, the comm project requires python >= 3.7 contrary to what is published (claims to support python 3.6)
I've seen that the traitlet dependency is used just in order to import this : https://github.com/ipython/traitlets/blob/main/traitlets/utils/importstring.py , which is pretty stable and easy
I suggest to just copy the method in comm package and remove the dep on traitlet all-together
what do you think ?
- 主要言語
- Python
- スター
- 12
- フォーク
- 19
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
似ている issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
canonical/paas-charm#368 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
tech debt
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
StevenBlack/hosts#3256 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
qualcomm/qai-appbuilder#275 ·