タグ

2023年2月15日のブックマーク (2件)

  • コードレビュー時のコメントの意図を明確にする

    コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか?」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物事が円滑にいかないみたいなことは発生します。 コメントする側からするとただ単に質問しているだけなのにどうして・・・と思いがちなのですが、コードレビューに慣れていない人はこういう重要度や意図がはっきりしないコメントをたくさん書いてレビュイーを困惑させがちというのはあるんじゃないでしょうか。 重要度や意図がはっきりしないコメントを書くのが常になってしまうと、GitHubでPRを出してレビューをしてもらってマージするという開発者が何度も何度も回していくワークフローに滞りが発生してしまう、もしくはそれどころ

    コードレビュー時のコメントの意図を明確にする
    moneycubed
    moneycubed 2023/02/15
    過去ひとりだけ、対面レビューで指摘したときに、言葉に詰まった上、今にも泣きそうな顔されたことある…。もうひとりのレビュワーと必死で宥めた。ので以降めちゃくちゃ気を遣っている。
  • 【追記あり】結婚、何から話し合えばいいんだ

    27歳女性、交際2年になる34歳の彼氏と「そろそろ結婚しようか」という話になっている。 が、細切れとはいえ2ヶ月近く話をしているのに、話が一向に前へ進まない。というか進め方が分からない。 私は入籍日を先に決め、そこから逆算して親挨拶やら引越しやらのスケジュールを立てていきたい。 一方で彼氏はどうやら先にの好みや生活習慣等の擦り合わせがしたいらしい。 いやまあ擦り合わせが大事なのは充分承知している。結婚してからどうしても折り合いつけられないことが分かるのも困るし。 でもそれは他の準備を進めながらでも出来るんじゃないかな…。正直今好きなべ物の話したって、と思いがち。 ざっくり擦り合わせている内容はこんなところか。 ・新居について 勤務地が同じなので、できれば近場に引っ越したい。SUUMOに希望条件入れて一緒にダラダラ見ている。 ・家事について 機械化歓迎とのこと。ドラム式洗濯機とホットクッ

    【追記あり】結婚、何から話し合えばいいんだ
    moneycubed
    moneycubed 2023/02/15
    うちはスケジュール立ててまず両家顔合わせした。二年付き合ってなお食の好みのすり合わせって必要なんだろうか…。あと34歳のひとの「高齢の」両親と将来的には同居、って将来どころかわりとすぐでは…?