タグ

Pull RequestとRequestに関するs1251のブックマーク (6)

  • 空コミット便利!git commit --allow-emptyでgitを使った開発フローを改善 - fukajun - DeepValley -

    何か? git commitのオプション--allow-emptyご存知でしょうか? これは、オプションの名前の通り空のコミットの作成を許可するオプションです。 通常変更がないとコミットが作れないようになってるので 空コミットを作るにはこのオプションを指定する必要があります。 add(もしくはrm)もしない(stageに何も載せない)で commitしたときの注意文には登場するので知ってましたが使ってませんでした。 最近、開発フローの中で使い道を思いついて使うようになったので紹介です。 その1 空Pull Request作れる プルリクって、基準になるブランチから変更されたコミットがないと作れないと思ってます。 でも、変更はないんだけどプルリクのcommentに変更の「概要」「目的」「ビジネスインパクト」「どの数値が改善するのか」など色々さきに書いておきたいこととかありますね。 考えてる内

  • レビューフレンドリーな開発のしかた - tomykaira makes love with codes

    2013-09-02 レビューフレンドリーな開発のしかた git dev 最近は多くのチームでレビューの習慣が定着してきました。おもにレビュアーとしての仕事を依頼されることもあります。 コミット・ブランチの作りかた一つでこのレビューのしやすさが格段に違ってきます。 自分が普段の開発でこころがけていることをまとめてみます。 前提 レビュイーとレビュアーの間に上下関係があるわけではないですが、レビュイーは多少手数が増えても、レビュアーのことを最大限配慮すべきです。 なぜなら、レビュイーはその機能の開発に集中して取り組んでいますが、レビュアーはすこし見るだけです。 なにかするとしたら、レビュイーがやったほうが時間も手間も少なくなります。 レビュアーはレビュイーよりも、変更について詳しくありません。 レビュイーは開発にいろんな部分を見てまわり、他のモジュールとの関連性や実装のこまかな意図を把握して

  • グリーとオープンソースソフトウェア - GREE with Open Source Software - | GREE Engineering

    インフラストラクチャ部の大場です。 今日から12月がはじまりました。みなさんは、今年やりたかったことはできたでしょうか。僕はといえば、やりたかったことが多くてあれもできなかった、これもできなかったとという後悔から目をそらすため、今日は11月32日かもしれないと強い意志で自己暗示をかけているところです。 さて、それはさておき今年を締めくくるのに相応しい企画として、これからグリーのエンジニアがクリスマスまで毎日このブログを更新していきます。題して『GREE Advent Calendar 2013』が始まります。 今回、口火を切る役としてグリーとオープンソースソフトウェアの関わりについてゆるくまとめましたのでお付き合いください。 グリーとオープンソースソフトウェアの関わり グリーはApacheやLinuxPHPをはじめ多くのオープンソースソフトウェアに支えられて開発をしています。 常日頃か

    グリーとオープンソースソフトウェア - GREE with Open Source Software - | GREE Engineering
  • GitLabにPull Requestを送るときの作法 - Qiita

    GitLabのPull Requestのコツがつかめてきたので、簡単に説明してみます。 とりあえずContributing Guidelinesをよく読みましょう。決まりを守らない人は嫌われて相手にされません。 あとは、Pull request guidelinesに従いましょう。 GitLabプロジェクトをフォーク フィーチャーブランチを作成 テストとコードを書く 複数のコミットが含まれる場合はsquashで一つのコミットにまとめる フォークにコミットをpush Pull requestを投稿 投稿したpull requestに関係するissuesを検索しpull requestのコメント欄に記述する 2.について、後日もう再度pull requestを送る場合は、ブランチを切る前に git remote add upstream https://github.com/gitlabhq/

    GitLabにPull Requestを送るときの作法 - Qiita
  • 開発時に出会いたくないパターン - Perl日記

    悩んだりうまくいかなかったり解決したり。だらだら書いた。 手作業症候群 とにかくなんでもかんでも手で確認・作業する必要があると思い込んでしまう病。 そりゃiOSアプリとかAndroidアプリとか最終的には実機確認は必須だけれども。 その前にやれることは多々あるはず。リグレとか。 あと「デプロイ職人」も不要にするべき。わかってる。 自動化できない要素を突っ込んでる方が悪いのだ。なんとかする。 masterブランチぶっこみ志向 masterブランチに直接コミットを重ねていくことにより開発速度をアップさせることができる。 ただし孤独な背水の陣を構えることになる諸刃の剣。 おとなしくtopic branchを切って作業するのが安心への近道であり王道である、 とか言ってたらみんなちゃんと切ってくれるようになった。めでたし。 チケットそっ閉じ症候群 来はリリースしたりデータを修正したりしてチケットと

    開発時に出会いたくないパターン - Perl日記
  • 自分の強みを生かすこと on Quipper - mizchi's blog

    今リリース前にしてはタスクがあんまりないのでブログ書いてみる。 Quipperに入社してから一ヶ月半ほど経過した。それで感じたことをあれこれ書いてみようと思う。 あんまり熱心に書くと前の会社に入ったばかりのことを思い出して恥ずかしくなったりするので、ほどほどにする。 エンジニア文化の共有 会社に入ってまず最初にやることは、エンジニア文化を共有することだと思う。 どんなマインドかは、僕より Quipper のスピード感 - @kyanny's blog とかを読んだ方が伝わるはず。 QuipperはOSS文化というかRuby文化的なものを強く志向している感じがあって、そこらへん馴染みやすかった。 入社してからやったこと まず前提として、ベンチャーなので整った教育制度などはない。(そもそも自分も研修など期待していない)。 エンジニアとしての文化を共有しているから、最初からすぐ仕事に入れた。具

    自分の強みを生かすこと on Quipper - mizchi's blog
  • 1