SlideShare une entreprise Scribd logo
1  sur  9
node.jsに勢いで
contributeした話
    @fukayatsu
発端


2012/04/15頃

Ubuntu12.04βにnode.js(v0.6.x)入れたぜー

http clientは動くのにhttpsは動かない???
いろいろ調べる


TLSのバージョン指定が怪しい

 (https://bugs.launchpad.net/ubuntu/
 +source/openssl/+bug/965371)

node.jsはその辺どうなってるんだろ

githubちょっと見てみよう
github

https://github.com/joyent/node

とりあえずtls.jsとか見てみる

...あれ、なんか重複してる行がある...

せっかくだし修正してpull requestしてみよう
はじめてのpull request
githubでfork

localにclone

作業用のbranchで修正

本家の修正に追随

pull request用のリポジトリに1つにまとめて
コミット
続き

同名のリモートリポジトリにpush

githubでpull request(https://github.com/
joyent/node/pull/3122)

“Thanks, good catch.”

Contributor License Agreementに署名してね
と言われる(http://nodejs.org/cla.html)
merged!


pull requestから約1時間後にはmergeされた

 https://github.com/joyent/node/commit/0f95a93


typoを修正してpull requestとかもよくあるみ
たい
反省とか
master branchからpull requestしてしまった

 参考) GitHubへpull requestする際のベストプラクティス http://
 d.hatena.ne.jp/hnw/20110528


コミットメッセージが適当だった

 参考) コミットメッセージの書き方 http://www.clear-code.com/
 blog/2012/2/21.html


修正部分のテスト書くと採用されやすいらし
い(そりゃそうだ)
まとめ

結局https clientの問題はopensslライブラリの
アップデートで解決した

お世話になっているものに貢献できて嬉しい

もっとgithub使おうと思った

Contenu connexe

En vedette

とりあえずAngular jsを導入してみませんか
とりあえずAngular jsを導入してみませんかとりあえずAngular jsを導入してみませんか
とりあえずAngular jsを導入してみませんか
Kenji Ono
 
たのしいNode.js
たのしいNode.jsたのしいNode.js
たのしいNode.js
ishiki-takai
 
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
Takuya Ueda
 
Node.jsではじめるサーバ構築
Node.jsではじめるサーバ構築Node.jsではじめるサーバ構築
Node.jsではじめるサーバ構築
AimingStudy
 
Node.jsで学生向けe-learningサイトを作った話
Node.jsで学生向けe-learningサイトを作った話Node.jsで学生向けe-learningサイトを作った話
Node.jsで学生向けe-learningサイトを作った話
Tessei Yoshida
 

En vedette (18)

EWD 3トレーニングコース#1 Node.jsとCacheの連携
EWD 3トレーニングコース#1 Node.jsとCacheの連携EWD 3トレーニングコース#1 Node.jsとCacheの連携
EWD 3トレーニングコース#1 Node.jsとCacheの連携
 
node.js
node.jsnode.js
node.js
 
Nodejs Introduction
Nodejs IntroductionNodejs Introduction
Nodejs Introduction
 
とりあえずAngular jsを導入してみませんか
とりあえずAngular jsを導入してみませんかとりあえずAngular jsを導入してみませんか
とりあえずAngular jsを導入してみませんか
 
たのしいNode.js
たのしいNode.jsたのしいNode.js
たのしいNode.js
 
phpck
phpckphpck
phpck
 
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
Angular jsとbootstrapでサクッと作るwebアプリ@html5勉強会 名古屋#2
 
大阪Node学園 七時限目 「ゼロからはじめるnode.js」
大阪Node学園 七時限目 「ゼロからはじめるnode.js」大阪Node学園 七時限目 「ゼロからはじめるnode.js」
大阪Node学園 七時限目 「ゼロからはじめるnode.js」
 
第4回鹿児島node.jsの会資料_内村
第4回鹿児島node.jsの会資料_内村第4回鹿児島node.jsの会資料_内村
第4回鹿児島node.jsの会資料_内村
 
Node.js アプリのプロファイリング
Node.js アプリのプロファイリングNode.js アプリのプロファイリング
Node.js アプリのプロファイリング
 
Node.jsではじめるサーバ構築
Node.jsではじめるサーバ構築Node.jsではじめるサーバ構築
Node.jsではじめるサーバ構築
 
Node.js+MongoDB in SPA
Node.js+MongoDB in SPANode.js+MongoDB in SPA
Node.js+MongoDB in SPA
 
UnityとBlenderハンズオン第2章
UnityとBlenderハンズオン第2章UnityとBlenderハンズオン第2章
UnityとBlenderハンズオン第2章
 
Grunt.jsを使った Expressの開発環境構築
Grunt.jsを使った Expressの開発環境構築Grunt.jsを使った Expressの開発環境構築
Grunt.jsを使った Expressの開発環境構築
 
UnityとBlenderハンズオン第8章
UnityとBlenderハンズオン第8章UnityとBlenderハンズオン第8章
UnityとBlenderハンズオン第8章
 
Java script初心者のためのnode.jsで学ぶドローン制御プログラミング
Java script初心者のためのnode.jsで学ぶドローン制御プログラミングJava script初心者のためのnode.jsで学ぶドローン制御プログラミング
Java script初心者のためのnode.jsで学ぶドローン制御プログラミング
 
Node.jsで学生向けe-learningサイトを作った話
Node.jsで学生向けe-learningサイトを作った話Node.jsで学生向けe-learningサイトを作った話
Node.jsで学生向けe-learningサイトを作った話
 
いよいよ始められる Java EEでのWebSocket #jjug #jjug_ccc #ccc_r21
いよいよ始められる Java EEでのWebSocket #jjug #jjug_ccc #ccc_r21いよいよ始められる Java EEでのWebSocket #jjug #jjug_ccc #ccc_r21
いよいよ始められる Java EEでのWebSocket #jjug #jjug_ccc #ccc_r21
 

node.jsにpull requestしてmergeされた話

Notes de l'éditeur

  1. \n
  2. \n
  3. \n
  4. \n
  5. \n
  6. \n
  7. \n
  8. \n
  9. \n