[B! Pull Request] wanijiのブックマーク

下から目線のコードレビュー - steps to phantasien

WEB+DB の新しいやつがちょっと前にでてます. コードレビュー特集だそうな. 時が経つのは早い. まだ次の原稿書いてないのに… そういえば前にコードレビューの話を書いた気がして, 見なおしたところかきかけだった. せっかくなので続きを書いてみることにします. といっても何書くつもりだったか覚えてないのでだらだらと. WEB+DB PRESS の特集は, 主にこれからコードレビューを導入したい人に向けて書かれている. 幸か不幸か私はコードレビューを義務付けれたプロジェクトで働いているため, 導入には苦労していない. かわりにレビューをちょろまかせない面倒はある. ある意味でコードレビューを <やらされている>. もちろんこの言い分は大げさだ. 必要性に異議を唱える気はない. ただ異議はさておき自分の意向とは無関係にコードレビューに参加している気分を書いた話はあまり目にしないので,

waniji 2019/07/18

とても良い記事

リンク

GitHub「完璧なプルリクの書き方を教えるぜ」 - Qiita

はじめにこの記事は How to write the perfect pull request - GitHub を和訳というか、意訳した記事です。ご指摘などありましたら大歓迎です！良いプルリクエストを書くには (原題 : How to write the perfect pull request) 会社が成長していくと、人もプロジェクトも様変わりしていきます。GitHubの中に私達が望む文化を育んでいくためには、我々が何を自覚してコミュニケーションするべきなのか分かってきました。私達のチームが最強であり続けるために、最近以下のようなプルリクエストのガイドラインを導入しました。プルリクへのコメント (原題 : Approach to writing a Pull Request) プルリクエストには目的を明記しましょう。たとえば… これは〜を調べてみるためのプロトタイプですこれは

waniji 2015/02/05

リンク

pplog開発のコードレビューから学ぶpull requestによる自律的行動とコミュニケーション - ppworks.jp

pplogの過去のポエムを複数単語で絞込できるようになりました。 pplogは、自身と向き合い想いを言語化するためのサイトだったりします。（色んな使い方があります）最新のポエムだけが他人に見えますが、自分の過去のポエムを見る機能があります。この過去ポエムは検索機能が付いているのですが、先日まで複数単語で絞り込むことが出来ませんでした。 pull requestが来た id: shootaさんからpull requestを頂きました。勝手にやった！まさにこれだ！と思いました。よし、コードレビューをしよう！命名に突っ込んだこれを見て思うところがありました。 search_word_arrays = params[:keyword].gsub(/　/," ").split() 私は言った for文にナニカを感じたし、Cぽい！ search_word_arrays = param

waniji 2015/01/27

リンク

“I wrote some code. I think you should maintain it.”

“I wrote some code. I think you should maintain it.” This is an important thing to consider when you, as a contributor, make a pull request, also as an author before hitting the merge button.