Patch brakes the argument order
まだ誰も着手していません。
評価
調査の方向性
fastlite/kw.py の 120 行目あたりから始め、パッチ適用後のメソッドシグネチャと元の update 呼び出しを比較します。.update(id, {"a": "b"}) を再現し、pk_items と updates が互換性のある位置を維持しているか、または明示的にキーワード専用になっているかを確認します。完了とは、パッチ適用後の API がこの呼び出しを壊さなくなり、エディターのナビゲーションとシグネチャ情報が誤解を招かないことです。
索引モデルが issue の本文から書いたものです。
説明
Hi!
This particular patch provides a subset of the original arguments. I can live with that theoretically, but it also swaps the positions of pk_items and updates. It took some time to find why my .update(id, {"a": "b"}) was broken 🙈 And the fact than my IDE (PyCharm) shows me hints about the original method and "go to definition" also jumps to the old one, doesn't help with such cases.
I understand that that "@patch"ing stuff was made for the sake of nbdev and it works in live environments. But such quirks with incompatibly changed method signatures make using of FastHTML in a Starlett's typical "hot reload" workflow pretty uncomfortable 😩
Maybe You can consider to make the args "keyword-only"? This will make the patching more robust.
- 主要言語
- Jupyter Notebook
- スター
- 232
- フォーク
- 19
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
AnswerDotAI/fastlite のほかの issue
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
AnswerDotAI/fastlite#76 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
AnswerDotAI/fastlite#72 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
AnswerDotAI/fastlite#67 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
AnswerDotAI/fastlite#66 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
AnswerDotAI/fastlite#63 ·
AnswerDotAI/fastlite の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
StellarState/SS-backend#712 ·
メンテナーはふだん 1 日以内に返信
-
New subtasks don't appear in the board card counter and disappear after switching views, until restart対応中かも @hatimpaloda88 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
litespeedtech/lscache_wp#1077 ·
メンテナーはふだん 1 日以内に返信
-
area/ysql kind/enhancement priority/medium status/awaiting-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
yugabyte/yugabyte-db#35578 ·
メンテナーはふだん 1 日以内に返信
-
String search treats % and _ as LIKE wildcards on Postgres対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug fhir-search ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
HasteHealth/HasteHealth#1250 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信