Patch brakes the argument order
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 48/100
Hướng nghiên cứu
Bắt đầu tại fastlite/kw.py khoảng dòng 120 và so sánh chữ ký phương thức đã được patch với lời gọi update ban đầu. Tái hiện .update(id, {"a": "b"}) và xác minh rằng pk_items và updates giữ các vị trí tương thích hoặc được chỉ rõ là chỉ nhận đối số theo từ khóa. Hoàn tất nghĩa là API đã được patch không còn làm hỏng lời gọi này và thông tin điều hướng/chữ ký trong trình soạn thảo không gây hiểu lầm.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Jupyter Notebook
- Star
- 232
- Fork
- 19
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của AnswerDotAI/fastlite
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
AnswerDotAI/fastlite#76 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
AnswerDotAI/fastlite#72 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
AnswerDotAI/fastlite#67 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
AnswerDotAI/fastlite#66 ·
-
tuplify where_args automaticallyĐang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
AnswerDotAI/fastlite#63 ·
Tất cả issue của AnswerDotAI/fastlite
Issue tương tự
-
Import/Export Data fails for a file name containing a backslash, and creates a stray file named EĐang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
pgadmin-org/pgadmin4#10672 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug/1-unconfirmed kind/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
prisma/orm#30859 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add: Ministerstvoidey.ruĐang mởcheck:passed logos:add
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Feature] 登录后不要自动展开所有模式及其子对象Đang mởenhancement qq-feedback
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày