あなたのプル要求を提出する前に避ける誤り
5851 ワード
初心者フレンドリーな問題の多くをブラウズした後issue にElement repo 私の注意を引きました.簡単に言えば、英語以外の言語のいくつかのテキストがホームページのボタンをオーバーフローします.あなたが私のような最初の貢献者であるならば、あなたは避けたいかもしれません: 投稿ガイドラインの削除 時代遅れのコードを押す
ファーストルック
これが問題です.
小さいCSSフィックス(以下に明らかにされる)が問題を解決するべきであるように思えます.私がレポをクローンするまで、トリッキーな部分が発展のためにプロジェクトを構築していると気がつきました.
私は、それがちょうどもう一つの反応プロジェクトであると思いました、それで、私はプロゲーマーの動きを作りました:どんなタイプのチュートリアルでもスキップしてください.結果として、私は間違ったrepoのまわりで2日pokingして無駄にしました.
なぜ貢献ガイドをスキップすべきか
私が貢献したオープンソースプロジェクトとは異なり、このプロジェクトは他の2つのreposを含んでいます.matrix-react-sdk and matrix-js-sdk . で説明したように
言い換えれば、この問題は
右のレポからコードを引いた後に、バグに責任があるコードの部分は簡単に特定されることができました.まず、バギーコンポーネントのCSSクラスをChrome devtoolsで検索できます.次に、Visual Studioコードのグローバル検索
こんな感じだった.
なぜあなたの支店を更新する必要があります
私がフィックスを持っていたならば、私はすぐに私のコードを私のgithubフォークに押しつけて、PRをしました.私は多くのコードを変更しなかったので、何が間違っていたかもしれませんか?
プロジェクトの共同研究者が私に何か本当に役に立つと指摘した.
さて、実際には2つの有用なものがあったが、それらの1つだけ私はCIを渡すことができました.別のコードの校閲者が残ったcomment 私が変更することによってさらに私のコードを改善する方法
最後にmy PR 完成し、要素プロジェクトのメンバーからのヘルプにマージされました.
修正後の同じページが適用されました.
どうやって修正しましたか
ファーストルック
これが問題です.
小さいCSSフィックス(以下に明らかにされる)が問題を解決するべきであるように思えます.私がレポをクローンするまで、トリッキーな部分が発展のためにプロジェクトを構築していると気がつきました.
私は、それがちょうどもう一つの反応プロジェクトであると思いました、それで、私はプロゲーマーの動きを作りました:どんなタイプのチュートリアルでもスキップしてください.結果として、私は間違ったrepoのまわりで2日pokingして無駄にしました.
なぜ貢献ガイドをスキップすべきか
私が貢献したオープンソースプロジェクトとは異なり、このプロジェクトは他の2つのreposを含んでいます.matrix-react-sdk and matrix-js-sdk . で説明したように
Development
guide にelement-web
repo、これらの2つのSDKがコードの貢献のために要素を正常に構築するために必要です.The idea of Element is to be a relatively lightweight "skin" of customisations on top of the underlying
matrix-react-sdk
.matrix-react-sdk
provides both the higher and lower level React components useful for building Matrix communication apps using React.
言い換えれば、この問題は
element-web
レポ、私が変更する必要があるコードは、実際にはmatrix-react-sdk
レポ.そのようなプロットツイストではなかったか.レッスンは学んだ:ガイドラインを読んで実際に多くの時間を節約できます.右のレポからコードを引いた後に、バグに責任があるコードの部分は簡単に特定されることができました.まず、バギーコンポーネントのCSSクラスをChrome devtoolsで検索できます.次に、Visual Studioコードのグローバル検索
Shift + Cmd + F
マックでShift + Ctrl + F
ウィンドウでは、それがCodeBaseに存在する場合は、その場所を明らかにすることができます.こんな感じだった.
.mx_HomePage_default_buttons {
margin: 60px auto 0;
width: fit-content;
.mx_AccessibleButton {
padding: 73px 8px 15px; // top: 20px top padding + 40px icon + 13px margin
width: 160px;
height: 132px;
...
}
}
なぜあなたの支店を更新する必要があります
私がフィックスを持っていたならば、私はすぐに私のコードを私のgithubフォークに押しつけて、PRをしました.私は多くのコードを変更しなかったので、何が間違っていたかもしれませんか?
プロジェクトの共同研究者が私に何か本当に役に立つと指摘した.
さて、実際には2つの有用なものがあったが、それらの1つだけ私はCIを渡すことができました.別のコードの校閲者が残ったcomment 私が変更することによってさらに私のコードを改善する方法
height
to min-height
.最後にmy PR 完成し、要素プロジェクトのメンバーからのヘルプにマージされました.
修正後の同じページが適用されました.
どうやって修正しましたか
.mx_HomePage_default_buttons {
display: flex;
...
.mx_AccessibleButton {
...
min-height: 132px;
...
}
}
それは500人以上の貢献者を持つこのような大きなプロジェクトの最初の受け入れPRだった.小さなフィックスでしたが、私の貢献の結果を見ながら、私は誇りを感じざるを得ませんでしたproject's change log . それにもかかわらず、一日の終わりには、私が学んだ2つの最も重要なことは、上記のミスです.Reference
この問題について(あなたのプル要求を提出する前に避ける誤り), 我々は、より多くの情報をここで見つけました https://dev.to/oliverpham/mistakes-to-avoid-before-submitting-your-pull-request-3nl8テキストは自由に共有またはコピーできます。ただし、このドキュメントのURLは参考URLとして残しておいてください。
Collection and Share based on the CC Protocol