Patch brakes the argument order
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 48/100
Direzione di ricerca
Inizia da fastlite/kw.py intorno alla riga 120 e confronta la firma del metodo con patch con la chiamata update originale. Riproduci .update(id, {"a": "b"}) e verifica che pk_items e updates mantengano posizioni compatibili o siano esplicitamente utilizzabili solo come argomenti denominati. Il lavoro è completato quando l’API con patch non interrompe più questa chiamata e la navigazione nell’editor e le informazioni sulla firma non sono fuorvianti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Jupyter Notebook
- Stelle
- 232
- Fork
- 19
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di AnswerDotAI/fastlite
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
AnswerDotAI/fastlite#76 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
AnswerDotAI/fastlite#72 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
AnswerDotAI/fastlite#67 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
AnswerDotAI/fastlite#66 ·
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
AnswerDotAI/fastlite#63 ·
Tutte le issue di AnswerDotAI/fastlite
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
percona/percona-xtradb-cluster-operator#2575 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Edit: Al Janoubiya TV SDApertacheck:failed feeds:edit
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
iptv-org/database#38385 · 1 commento ·
I maintainer di solito rispondono entro 6 giorni
-
bug qq-feedback
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno