Suggestions
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- go
- Lĩnh vực
- documentation
Hướng nghiên cứu
Bắt đầu bằng cách xem README của repository và phần về di chuyển lỗi được liên kết trong docs/RFCS/20190318_error_handling.md, sau đó kiểm tra cách exthttp và extgrpc hiện đang được trình bày. issue chứa nhiều đề xuất riêng biệt, vì vậy trước tiên hãy xác định một thay đổi tài liệu duy nhất đã được thống nhất. Công việc được xem là hoàn tất khi có một bản cập nhật tập trung với các ví dụ hoặc liên kết phù hợp với phạm vi đã chọn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
For @knz
I was looking through the docs trying to find out how to register a migrated error, and it took me a little bit. So I wanted to suggest how to make that more visible, and then I thought while I'm at it, a couple more suggestions:
- First, the docs on migrating/renaming an error are a little buried and could perhaps be linked in the README. You talk about forward compatibility in the README, which I understand in part to involve forwarding errors through unaware intermediaries but from the name it sounds like error migration is part of it, so I was surprised how much I had to dig to re-find this data.
-
RegisterTypeMigrationcould perhaps be part of the "forwarded methods" so it shows up in the main package, looks like I currently need to importerrbaseto get at it. - The README could use a sample of the ultra-awesome stack trace format!
- The compatibility table is a bit intense to be at the top of the README. I remember back when I first came across this package, it was a little confusing—my takeaway was, "wow, this thing has lots of features I guess". I feel like you might be better served by having the top of the README dedicated to a simple bullet-point list of features and some code samples showing what it looks like in practice.
- Could have a godoc badge up at the top of the README:
- Perhaps rethink how the data on custom errors is provided. The information is pretty dense and unstructured; I would suggest more subheadings for specific tasks like "Custom Leaf", "Custom Wrapper", "Over the Network", or some such. Also perhaps consider putting the writeup in its own document because there's a lot to the subject and it can be overwhelming to someone landing on the main page of the repo.
- This is subjective, but I think in general the README could express more through code samples:
- The section on Available Error Leaves could show little fake error scenarios, using that applicable function, and then retrieving the data, with additional details in comments.
- The section How to Use could perhaps be shown in code.
- I question the necessity of API (not constructing error objects), since Godoc is probably better suited for this anyway.
- I don't think the
exthttppackage is mentioned as a useful tool, more as an example for when you're building your own error types, but I think it adds value. Alsoextgrpcisn't mentioned, although that one's entirely my fault because I never added docs for the work I did 😉 - I feel like the project deserves a cool logo! (😄 ) Maybe a variation of the cockroachdb logo?
I'm aware that I'm armchair-quarterbacking and should probably put a PR where my mouth is, but I'm in the middle of other things and just felt compelled to jot down some notes. Interested to hear your thoughts!
- Ngôn ngữ chính
- Go
- Star
- 2.5k
- Fork
- 75
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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 cockroachdb/errors
-
Errors returned from Join are incompatible with errors returned the standard lib's errors.Join Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 52/100
cockroachdb/errors#162 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
cockroachdb/errors#156 · 20 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
cockroachdb/errors#155 · 1 bình luận · 1 reaction ·
-
move subpackages to internal Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
cockroachdb/errors#146 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
cockroachdb/errors#145 · 1 bình luận · 1 reaction ·
Tất cả issue của cockroachdb/errors
Issue tương tự
-
textual definition
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
geneontology/go-ontology#32653 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
-
needs design
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Priority/High ready-for-agent Severity/Major Type/Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100