タグ

ブックマーク / ppworks.jp (3)

  • OSSに貢献する気持ちを育てよう - ppworks.jp

    会社のブログに「database_rewinderにpull requestを投げた結果」という記事を書きました。 rails の良い所の一つに、豊富なgemによるエコシステムが優れているという点があります。 gemの多くはOSSとしてgithubにそのソースが公開されています。githubにはPull Requestがあります。Pull Request、いい響きですね。世界の名だたるエンジニアの作ったソースコードにPull Requestが送れるだなんて、なんて感動的なのでしょう。 railsを使う企業であれば、そのエコシステムに入る覚悟が必要です。OSSへの貢献が自然とできるような環境で仕事出来る事ということは、railsを使う覚悟のある会社を見定める際の一つの基準になるかも知れません。 OSSに貢献する気持ちとは何でしょう、エコシステムの一員として当事者意識を持って関わるときの喜びを

    OSSに貢献する気持ちを育てよう - ppworks.jp
  • クソコードと呼ばない - ppworks.jp

    新しい現場にはいったときに心がけていること、クソコードと呼ばないこと。 誰かのコードを読んでいるとそりゃまあクソコードを見つけることがある。その時どう立ち向かうかという精神論の話。 例えソレがそうであってもソレを口にするとネガティブが蔓延する。思ってもイイ、でも言ってしまってはならない。あるフェーズに置いては必要だった し、現に動いていて価値を提供している のだ。あるべき姿を叫ぶの簡単だ。あるべき姿を見ているなら行動しないといけない。見つけたらリファクタだ。出来るところからやるんだ。 Shut the fuck up and write some code & 許可を求めるな Pull Request せよ— 🌈KOSHIKAWA (@ppworks) 2014年5月23日 クソはクソと言える空気や文化は大事。良くないものを指摘できるようにはしたい。口の前に手を動かそう。プログラマーなら

    クソコードと呼ばない - ppworks.jp
  • コードレビューをしよう - ppworks.jp

    Sendagaya.rb #86 - Sendagaya.rb | Doorkeeperに参加してコードレビューについて思うところをみんなでディスカッションしたので、自分の思いをまとめておきます。(ポエム) ソニックガーデンの @mah_labさんのスライドにある7つの秘訣に同意です基。あと、github上のプルリクでコードレビューするという前提でまとめます。 1. レビューの観点を明確に いま何の話をすべきかという状況を判断して、どの観点で物を言うのかっていうのはコードレビューだけに限らず打ち合わせ時にも大事ですね。今この観点で議論を進めますよ!ということを表明することで齟齬が防げます。 観点が明確だと議論のテーマが明確になるので、コードレビューで修正すべき点も分かりやすくなりますね。 コーディング規約の観点 セキュリティの観点 メンテンスの観点 設計の観点 リリース前にこれで出来るレ

    コードレビューをしよう - ppworks.jp
  • 1