続: たのしくなるコードレビュー
を書いたんですけど、書ききれなかった事があるので補足します。
レビューを促す方法について
レビュー待ちがたまりがちになるじゃないですか。*1
ならないですか?
なるのでその対策として、任意の時間になるとレビューを促すリマインダをセットしていた時期があります。
でもこれは次のようなところが良くなくて辞めました。
- タイミングが悪いと集中が切れる
- いつもリマインドしてくるから段々意識が向かなくなる
今は、出社したらまずレビューをして、それから開発に入るようにしています。
こうすることで開発に入る前のウォーミングアップになるし、忘れたりスルーしてしまうことがないです。
レビューが必要そうなプルリクが出てない時は、メール処理などの適当なシャローワーク*2をしてから開発に入ります。
この方法が自分には合っていたため定着してきています。
静的解析ツールについて
これ
lintもかけずにコードレビューとな pic.twitter.com/SnOlTeMouz
— こまたつ (@k0matatsu) 2017年9月22日
Lintもかけずにコードレビューするのは自傷行為*3なのでやめよう。
デフォ設定ではだいたい使い物にならないと思うので、チューニングする必要がある。
このへんイニシャルコストがかかるけど、長い目でみると余裕でペイできるのでちゃんとやろう。
チューニングの方針について、既存のコードベースに導入する場合、一旦手元でかけるとめっちゃエラー/警告が出るはず。
全部直してクリーンな状態を作るのはすごくモチベーションがいるし現実的ではないので、対応しきれないものに関しては一旦黙らせておくのが良いと思います。
開発が一段落付いたタイミングとか、お盆とか年末とかでちょっとづつ対応していってという方法がおすすめです。
とにかくLintをかけるんだ。
余談
社内でテックライフアワードというのがあってブログがバズると好きな本をもらえるんですよ。
普段あんまりブログとか書かないので300ブクマ超えて有頂天になった私はこれアワードワンチャンあるなと思ったんですよ。
調べたら3位で500ブクマ超えててお前ら化物かよと思いました。
完全に負け惜しみなんですけど、ブクマがすべてじゃないし、「コードレビューするのつらいな」って思っている人の役に立ったらうれしいです。
ダンケルク
予告編見た感じで見る気はなかったんですけど
同僚が良いというので見てきました
撤退戦の華は殿でしょ!と思っているんですけど、イギリス視点なので殿を務めた仏軍はほとんど出てこないです
民間船舶の徴用のためにこんなに頑張りました!みたいな描写がなかったのも物足りなくて
いったいこの映画はいつカタルシスを与えてくれるのかと思っていたら重い気持ちのまま終わってしまい
努力賛美が染み付いてしまっているのが良くない
エンジン停止したスピットがメッサー落としたときは流石に盛り上がったんだけど「すげー!」と言うよりは「どんだけ〜〜〜」という気持ちだったので流石にあれはないんじゃないの
なんなの
わたし向けでは無かった
プロメテウス
http://www.foxmovies.jp/prometheus/
エイリアンの前日譚らしく、おなじみのスリラー展開でバッタバッタと登場人物が死んでいきます 流石にこれだけ長くやっているシリーズだと序盤数分で誰が最後まで生き残るかわかってしまうんだけどまあ予想通りの展開になる 登場人物の皆さんはまじで頭悪いので何が起きてもそんなバカなと思ってはいけない 思ってはいけない24時なのである
人間の始祖っぽい生命体がなんとも言えぬデザインでキモく、外骨格がなんとなく巨神兵っぽい もっとプレデターっぽかったら良かったのに...
シャーリーズ・セロンが出て来るんですけど、イーオン・フラックスの頃と全然変わってなくて こいつがロボットかよと思うんだが劇中でも指摘されていて笑う
エイリアン:コヴェナントを見る予定の人はあらかじめこれを見ておくと「あ〜〜〜〜〜〜」となるのでおすすめです
予習にどうぞ、
ジョン・ウィック:チャプター2
公式の予告編が悪ノリって感じがして楽しそう 1に比べるとグラップリングの比重が増えていて派手さはあまりないけれど見応えあるアクションになってた CARっぽいガンアクションもより洗練されていて練習頑張ったんだなあ
アレスがまた良くてなんで敵なんだ味方になってくれ頼むという気持ちでいっぱい ソムリエに武器選んで貰うシーンは良かったんだけどその前の描写との整合性が取れてない気がして腑に落ちてない まあ面白かったからいいけど
次回作も期待してます
おすすめ神社情報
お疲れ様です、私です。
そろそろ初詣の季節ですね。
忘年会でオススメの神社を聞かれたので私のオススメの神社を紹介します。
このエントリーには神様をあんまり敬ってない感じの表現が出てきますので信仰ガチ勢の人は見ないほうが良いと思います。
神道的には氏神を参拝するのが正しいんですが、近所に神社がない人とか氏神とか神道とかどうでもいい人はいい神社に行きましょう。
なお、生まれた土地の氏神を産土神、ご近所じゃない神社の神様を信仰する人を崇敬者と呼ぶそうです。
それでは私のオススメの神社を3社紹介します。
福徳神社
日本橋らへんにある神社で異様にきれいです。
夜間はライトアップされており花のさく時期はとても厳か。
見た目は新しいがだいぶ昔からある由緒ある神社です。
主祭神はウカノミタマノミコト、稲荷です。
ちなみにウカノミタマは女神とされている。ケモナー歓喜
金銭面のご利益があって宝くじのCMとかにも出ていた時期がある。
御朱印は日によって特別な物が貰える事がありコンプリートしたくなります。
立地柄参拝後に初売りとか行けそうな感じがする。
社殿以外も近代化が進んでおり公式ツイッターアカウントもあるぞ
干支土鈴クソ可愛い
ちなみに、新年に授与致します干支土鈴も奉製致しました。芽吹きの模様の入った卵形?の鶏です。御初穂料は一体800円。年明けから授与致します。 pic.twitter.com/qkvNxXXE1e
— 福徳神社 (@fukutoku_mebuki) 2016年12月13日
調神社
浦和にある神社でツキノミヤと呼ばれている。
月待信仰らしく狛犬がウサギ、手水舎の水が出てくる所もウサギ、噴水もウサギ、とにかくウサギ。
ウサギ好きは一度は行った方が良いと思いますが結構リアル志向なので可愛さは控えめです。
でかいですが鳥居がないタイプの神社なので鳥居大好きマンの人は別の神社に行きましょう。
主祭神はアマテラスオオカミ, トヨウケビメノミコト, スサノオノミコトです。
主祭神なのに3柱おり、どれが主なの?って感じですが観光協会とかのホームページを見てもわかりませんでした。三連複的なやつなのかな
ツキという名前から勝負事にご利益があるそうです。真偽は定かではない
隣にでかい公園があって家族のいる人は子供を遊ばせておく事が可能
境内にゼルダの伝説シリーズに出てきそうな剣を携えた人の石像があって微妙に怖いので行った人はぜひ探してほしい。
来年はワンチャン狙っていきたい人たち、如何でしょうか
氷川神社
氷川神社いっぱいありますけど大宮のやつです。
戦艦武蔵の艦内神社も氷川神社です。提督の皆さんは参拝しましょう。
全体的に手入れが行き届いておりたのしく観光できる。
鶴岡八幡宮とかは階段登るのがキツいんですけど大宮氷川神社は縦の移動が無いので最高。
ただし正月の参拝者が異常に多くて死ぬので初詣には覚悟が必要。
せっかくなので氷川参道を通っていった方が楽しめます。
主祭神はスサノオノミコト, イナダヒメノミコト, オオナムチノミコトの三連複です。
それぞれ男体社、女体社、簸王子社に分かれており三社同格ということになっている。
あのへん一体を出雲族が開拓したらしく、オオナムチノミコトが居るのはそのためっぽい。
元の氏神はアラハバキとされているがアラハバキ自体が謎なので無視します。
多分出雲族の人たちは治水技術に優れていたため荒川近辺の開拓が出来たのではないかと予想する。
ご利益はいろいろです。
年始からフェス感味わいたい人にオススメ
というわけで3社オススメしました。
北区に住んでいるので北側が多いです。来年は南側も参拝していきたい。
最後にひとつだけ言っておくが後ろの方から賽銭を投げるのは危険なので辞めよう。
あなたが神様だったとして金属片投げつけられて嬉しいですか?
当たるとすごく痛いし年始から神社で嫌な気持ちになりたくないのでマジで頼む。
それではまた、