私はウンザリしています。 「○○対応」は曖昧なのでやめてください。「○○を修正した」の方が直接的です。 こんな指摘を新人が入ってくるたびにコードレビューやドキュメントレビューで繰り返しています。どうも、プログラマー（と言うか理系？）には独特の言語文化があり、みんな同じような分かりにくい表現をしてしまうようです。 「レビューを依頼する前にこれを読んどいて！」と言える記事なり本なりがあれば良かったのですが、良いものが見つけられなかった（ご存知なら教えてください）ので、とりあえずレビューでよく指摘する日本語の文章の問題点や変な表現ポイントを列挙しました。 なお「コメントは必要十分な量を書く」「チケット番号やWikiのURLを書く」といった、良く知られた・日本語に限定されない話題は省略しています。 1. 俗称を使わない・造語しない 製品名・機能名などは正式なものを使いましょう。大抵はUIなりマニュ
PySpa統合思念体です。 某チャットで、「今時のOSSのプロジェクト管理とかのベストプラクティスが書いてある本ないかな、陳腐化早そうだしないか」みたいな話題が投入されました。その中で、エキスパートPythonプログラミングとか、Pythonプロフェッショナルプログラミングとかは思い出して紹介したけど、他の人からはShip It、Manage It、Release It三部作とか、達人プログラマーとかも出てきました。 このあたりの源流を辿ると、そういえば今流行ってる開発の源流としてはエクストリームプログラミングの開発系のプラクティスの遺伝子を受け継いでいるのが多いよな、そういえば当時から見て今ってどう変わっているのかな、という話題に。せっかくなので20年前を思い出しつつ、当時と今でどういう風に変わってきたのか適当にまとめてみます。 20年前の状況 XP白本こと、eXtreme Progra
よいコミットメッセージとはどういうものだろう？ - chiastolite’s blog に対するアンサーソングです chiastolite.hatenablog.com 必要であればWhyを補足するためのContextを書く 元エントリだとコミットメッセージの話だったけど僕はPRで実践してもいいと思います。*1 起点となるURLを明記する レビューで指摘されたのであればPRのコメントのURL issueのURL アプリケーションエラーであればSentryなどのエラートラッキングシステムのURL GitHub外のタスク管理ツール（例：redmineやBacklogやTrelloなど）に修正概要が書かれているのであればそのURL チャットのやり取りが起因ならSlackなどのパーマリンク 公式ドキュメントのURL CIがコケているのであればCIのビルドのURL チーム開発していればコミットやP
はじめまして！２年目の駆け出しSEです。 先日まで新人研修に関わって、2年目のくせに新入社員たちに偉そうに教えていた@takkii1010です。 私が関わっていた研修では、先輩社員1人で新人複数人を運営するチーム開発が行われました。 教わる側から教える側（！）に立った時に、どうやって自分の実力不足を補いつつ新人にも成長してもらえるか、どうチームを運営していくか悩んだのでここに簡単に残します。 この記事のメインターゲット 新人教育に携わる人 「ふふっ」と少しだけ笑顔になれるような研修にしたい人 研修概要 新人7名＋リーダー1名（自分） チームでWebアプリケーションの開発を行う 新人それぞれが異なる機能を実装する つまり、質問内容がバラバラになる可能性大 チーム開発への課題 「画面遷移しないんですが！助けて！」という質問はしてほしくない（もう一歩進んでほしい） せめてスタックトレースみてほし
「気に入らないコード」をレビューする際にRed Hatのエンジニアはどうしているのか：10個のヒントとは？ プロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合はどうしたらよいだろうか。Red Hatのソフトウェアエンジニアが、コードレビューを行うに当たって念頭に置くべき10のヒントを解説した。 Red Hatでソフトウェアエンジニアを務めるデビッド・ロイド氏は2019年7月8日（米国時間）、コードレビューを行うに当たって念頭に置くべき10のヒントを同社の開発者向け公式ブログで解説した。プロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合に役立つ指針だ。コントリビューター側としても参考になる。 これらのヒントは、客観的で的を射たレビューを行い、プロジェクトとその参加者を前進させるという観点からまとめられている。 （1）
このエントリは全9回を予定する18卒新人ブログリレーの第4回です！ 今回は、SUUMOのモバイルアプリ開発の現場から、企画と開発が一体となったチームでの開発プロセスについてご紹介します！ はじめに はじめまして。リクルートテクノロジーズ新人の三田涼介です。現在は不動産検索サービスSUUMOのAndroidエンジニアとして働いています。 今回はSUUMOアプリチームの開発プロセスについて紹介します。 SUUMOは巨大なサービスでありながら日々高速に改善を行っており、週1回以上の頻度でiOS、Androidの各OSでリリースを行っています。このスピード感を実現するために、開発プロセス自体も日々磨き込みが行われており、私が配属された3ヶ月前にも開発プロセスに大きな変化がありました。 この記事では、私が新人としてキャッチアップしていく中で学んだ、 なぜ開発プロセスを選ぶことが必要なのか どうやって
イベント概要 先日、アジャイルの本質について語るイベントが日本橋で開催された イベントのテーマは「本質を理解しないままのアジャイル開発に挑む危うさ」 アジャイル3怪獣 と呼び声の高い (?!) お三方がアジャイル開発の本質を激辛トークで深掘り 会の後半では、参加者を交えたディスカッション（フィッシュボール形式）も行われた イベント詳細はこちら：https://connpass.com/event/96750/ 登壇者 やっとむさん「アジャイルってなにが美味しいの？」 きょんさん「目的論からのアジャイル～手段を目的化すること～」 ryuzeeさん「アジャイル開発でコールド負けしないための5つのポイント」 感想を一言で言うと タイトル通り、内容が本質すぎて辛かった 終始「ぐぬぬ…」となる胸熱展開のオンパレードでした 「アジャイルってなにが美味しいの？」 by やっとむさん 登壇資料 そもそもア
先週金曜日に開催されたMinami Aoyama Night #6で「よりよいレビュー環境を求めて」という発表をしたので資料を公開します。以下、発表内容を文字に書き起こしたのでよろしければお読みください。 tl;dr Disclaimer なぜコードレビューをするのか はてなブログの開発フロー 避けたいPull Requestのパターンと、それを回避するための戦略 一発入魂パターン 大会場パターン 数珠繋ぎパターン バザールパターン レビューしやすいPull Requestを作る戦略 実装にあたって必要なタスクを事前に細かく分割してまとめておく Pull Requestは細かくmasterにマージする レビューを取り巻く環境を工夫する レビュアーの偏りを可視化する アルバイトスタッフにもレビューをしてもらう まとめ tl;dr レビューはソフトウェアの質を担保する重要なファクターといえる
https://www.flickr.com/photos/visualpunch/7477997812「問題対私たち」のチームになっていくには？チーム立ち上げのころは相互参加−相互貢献−相互尊重−相互信頼がまだチーム内で醸成されていない状態であることが普通です。このタイミングでは、チームメンバーは些細な困りごとを安心して話すこともできず、仮に本丸の取り組むべき問題をチームで見つけても、チームメンバーが協力して前向きに取り組むことできないことがあります。チーム内の人と人とのコラボレーションの質が低い状態の場合は、ふりかえりの場が、無関心、他責、厄介事の押し付け、一方的な説得、ディベート（言論で打ち負かす）の言動を取りがちで、機能不全に陥ります。 では、どうやって、本丸の重要なテーマの問題にも「問題対私たち」で取り組める相互信頼のあるCool&Sexyなチームになっていくのでしょうか？ KP
今日，社内勉強会で話す機会があり，過去1年間を振り返りつつ「プロジェクトをリードする技術」というタイトルにした．今回は参加者がエンジニアだけじゃなく，ビジネスチームのメンバーもいたため，できる限り，技術的な用語を使わないようにした．質疑応答とディスカッションもあり，1時間非常にワクワクした時間だった． 関連する領域 僕がプロジェクトをリードするときに意識しているのは，スクラムなど特定のプラクティスに依存しすぎないことで，チームの特性によって，関連する様々な領域からプラクティスを集めている．ザッと挙げるだけでも，こんなにたくさんある． チームビルディング ファシリテーション マネージメント 3.0 アジャイル (スクラム / カンバン / XP) 組織論 育成 心理学 メンタリング プロジェクトマネジメント 資料 過去1年間に取り組んだことを全て詰め込んだ！プレイングマネージャーとして頑張っ
Deleted articles cannot be recovered. Draft of this article would be also deleted. Are you sure you want to delete this article? ex-mixi Advent Calendar 2017 - Qiita 9日目をいただいた @kkakizaki です。 株式会社ミクシィの在籍期間は2008年から2016年までで、 QAマネージャーとして、次々に巻き起こる品質保証上の課題を何とかするお仕事をしていました。 現在は株式会社サイバードにて、同じく品質保証部門の長として、 品質保証だけではない様々な課題解決に勤しんでおります。 折角の ex-mixi なので、今回は この記事 の続きです。 開発現場に根強く残る8個のバグレポートアンチパターン 前回の記事は2012年に書か
属人化の理由 個人の問題 手抜きやバグを隠す たとえば、仕様書外の動作を実装し、それをプロジェクトで利用する 解雇されないための保険的行動 チームの問題 マニュアルを作る文化の欠如 他人のタスクに対する無関心 他人の監査なしにプロジェクトを更新可能 どうやって属人化を避けるのか 間違った対策 ○○さん以外にもマニュアルなしで操作できる人間を育成 育成した人が全滅すればやっぱり同じ状況 全員がすべてのプロジェクトに精通するとかはムリ 正しい対策 モジュールごとに仕様書を用意 間違って使うことが難しい仕様とする 即ち、仕様書を読まなくてもある程度正確に使える お互いにコードレビューさせる 具体的にはどうすれば良い？ テスト・仕様書・利用例 テストは仕様書のベースとなる 仕様書を見れば、深い動作がわかるようになる 仕様書を読まなくても、利用例を見れば使える 全員がテストできる環境を作る 前提条件
弊社では、案件とは関係のないプロジェクトでも業務時間中にみんなにコードレビューを依頼できる時間が確保されています(参加は任意)。案件のコードレビューを依頼したり、ちょっとした個人の制作物を見てらったりと使い方は色々です。 先日、TypeScriptの練習にQiitaのAPIを叩いていて記事を表示するブログウィジェットを作成しました。このウィジェットのレビューを依頼したところ、クラスの設計について具体的な指摘と、それに対する改善を経験できたのでこの記事に記載します。 今回作ったQiitaWidgetの要件 Qiitaの公式APIV2から記事とユーザー情報を取得し、HTMLテンプレートに表示する 投稿の合計いいね数を算出するために、あるユーザーの投稿を全件取得する (このために複数回リクエストの送信とレスポンスデータの結合を行う) パラメータによってユーザー、いいね数によるソート、表示件数、ラ
GitHubを中心とした開発プロセスの課題管理編です [milestone] Issueの対応期限を明示する 適切な対応期限を設定して積み残さない様にしよう 使い方 ここからMilestoneの作成・編集が出来るぞ 開始日と終了日とタイトル、任意で説明を記述する 作ったMilestoneはLabelやAssigneesと同じようにIssueやPullRequestに設定できる Milestoneの一覧画面では、関連するものの数や、進捗を見ることが出来る Label等と同様にIssue一覧画面ではMilestoneでのフィルタリングも出来るので、Issue等の優先度を理解しやすくなるぞ [issue link, pull request link] IssueやPullRequestを紐付ける 関連するIssueやPullRequestは紐づけておこう どうして？ Issueで議論をして、対
こんにちは！サービス開発部でAndroidアプリの開発をしているこまたつ（@k0matatsu）です。 みなさんコードレビューしていますか？ 最近ではとりいれているチームも多いと思いますが、良い効果をもたらしてくれる一方で、負荷の高い作業でもあります。 また、コードレビュー自体に馴染みの薄かった人はなにをどうしたらいいのか難しいですよね。 同僚から得たアドバイスと自分なりのノウハウをあわせて、コードレビューの指針を考えていたので公開してみようと思います。 前提として、クックパッドではGitHub Enterpriseとプルリクエストを使った開発プロセスを採用しています。 また、コードレビューの前には自動テストと静的解析ツールによる単純なフォーマット、コードスタイル、頻出バグのチェックは行われているものとします。 静的解析による機械的なチェックはコードレビューよりも低コストで有効な方法ですの
はてなのアプリケーションエンジニアのid:shiba_yu36です。社内技術勉強会で「新機能作成時に開発ブランチに細かくmergeしていく戦略」という発表をしたので、資料を公開します。 speakerdeck.com 以下、簡単に文字でまとめておきます。 戦略 ユーザーに新機能が見えないようにする工夫をし、新機能のbranchもどんどん開発ブランチにmergeしていく mergeされたものは随時本番にリリースされるが、ユーザーに見えない工夫をしているので問題なし PRは可能な限り細かくする 機能が完成したら最後にユーザーに新機能を見えるようなPRを作り、mergeしてリリースする なぜこの戦略を使うのかいろんな失敗を経験して、この戦略を最近使っている。 失敗パターンその1: 巨大PRパターン 失敗パターンその2: 新機能リリースブランチパターン 失敗パターンその1: 巨大PRパターン 新機
Deleted articles cannot be recovered. Draft of this article would be also deleted. Are you sure you want to delete this article? はじめに 今期から、クライアント企業の開発案件を抜けて、 企画、マーケティング含めた自社サービスのディレクションをすることになりました。 いわゆる社内外のサービス全般のなんでも屋さんでもあるため、 決まったタスクだけでなく臨時の依頼が割り込んだり、タスク管理が煩雑になりがち。 かといって、現在たった2名のチームで進めているので 堅苦しいタスク管理表とかはいらんだろ、、、(･Д･｀) そこで、Trelloを導入してみたので事例としてまとめておきます。 普通の開発プロジェクトで使う管理方法とはちょっと違うので、悪しからず。 Trelloとは
社内の方と雑談する中で、「属人化」に対する一つの考えがまとまったので書きます。「属人化排除のやり方」ではなく、「属人化排除を考える時期」についてです。 「属人化排除」というアプローチで感じたこと 「属人化排除」は大事だと思います。人的SPOFをなくすのは強いチームを作る上で重要です。しかし最近、複数のプロジェクトに関わっていく中で、「プロジェクトの時期によって属人化排除の優先度は変わる」ということを感じました。 あるプロジェクトXがAさんとBさんを中心に回っていました。そこは結構な属人化が起きていて、ほぼAさんとBさんしか仕様がわからず、他の人がコードを触るのは難易度が高くなっていました。当初は「このままで大丈夫かな」と思っていたのですが、よくよく考えると「あぁ、今はそういう時期だ」と感じました。この感覚を今回は言語化しようと思います。 プロジェクトの「加速時期」と「速度を落とさない時期」
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
