Review forge lint "unsafe-typecast"
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- solidity
- Lĩnh vực
- blockchain
Hướng nghiên cứu
Bắt đầu với các phát hiện về unsafe-typecast từ pull request tiếp theo #290 và kiểm tra từng trường hợp cắt ngắn được báo cáo trong codebase. Quyết định xem đó là có chủ ý, vô hại hay đáng để sửa, không thêm các guard cho những trường hợp bất khả thi hoặc thay đổi chữ ký; quá trình review hoàn tất khi các phát hiện đã được xử lý và lint vẫn bị vô hiệu hóa nếu còn lại hơn hai false positive.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
My brief examination found these were all false positives, but as a follow-up to https://github.com/FilOzone/pdp/pull/290, we want to look over them to see if any are worth fixing. In some cases these truncations are intentional. In other cases they are harmless.
Do not add guards against impossible situations.
Assume that parameters supplied by the owner will be reasonable, though certain footguns may still be worth protecting against, depending on the impact.
Do not change function signatures.
Prefer to leave this lint disabled if there are more than 2 false-positives remaining in the codebase rather than proliferating forge-lint: disable-next-line(unsafe-typecast).
- Ngôn ngữ chính
- Solidity
- Star
- 20
- Fork
- 18
- 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
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 FilOzone/pdp
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
Release process improvement ideas/observations from pdp v3.5.0Có thể làm lại được @BigLep đã nhận 81 ngày trước và không có pull request nào đang mở. Đang mở
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 28/100
Issue tương tự
-
Log filters reject `"fromBlock": null` (and `toBlock`/`blockHash`) instead of treating it as unsetĐang mởL1 rpc-api
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
lambdaclass/ethrex#7530 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
yegor256/erc20#218 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[BUG]: Oracle with rewardBps above 1e5 can never accept a funded price submissionCó thể đã có người làm @kansalakshat đã nhận 1 ngày trước. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
StabilityNexus/OrbOracle-Solidity#27 · 1 bình luận ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100