Suggestions
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- ドキュメント
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- go
調査の方向性
まずリポジトリのREADMEと、docs/RFCS/20190318_error_handling.mdにリンクされているエラーマイグレーションのセクションを確認し、その後、exthttpとextgrpcが現在どのように示されているかを調べます。issueには個別の提案が多数含まれているため、最初に合意された単一のドキュメント変更を特定します。完了条件は、選択した範囲に合った例またはリンクを含む、焦点の絞られた更新です。
索引モデルが issue の本文から書いたものです。
説明
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!
- 主要言語
- Go
- スター
- 2.5k
- フォーク
- 75
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cockroachdb/errors のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 52/100
cockroachdb/errors#162 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
cockroachdb/errors#156 · リアクション 20 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
cockroachdb/errors#155 · コメント 1 件 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
cockroachdb/errors#146 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
cockroachdb/errors#145 · コメント 1 件 · リアクション 1 件 ·
cockroachdb/errors の issue をすべて見る
似ている issue
-
textual definition
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
geneontology/go-ontology#32653 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
-
needs design
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
Priority/High ready-for-agent Severity/Major Type/Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100