SlideShare uma empresa Scribd logo
1 de 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使おうと思った

Mais conteúdo relacionado

Destaque

とりあえず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
 

Destaque (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
 

Último

Último (10)

新人研修 後半 2024/04/26の勉強会で発表されたものです。
新人研修 後半        2024/04/26の勉強会で発表されたものです。新人研修 後半        2024/04/26の勉強会で発表されたものです。
新人研修 後半 2024/04/26の勉強会で発表されたものです。
 
論文紹介: The Surprising Effectiveness of PPO in Cooperative Multi-Agent Games
論文紹介: The Surprising Effectiveness of PPO in Cooperative Multi-Agent Games論文紹介: The Surprising Effectiveness of PPO in Cooperative Multi-Agent Games
論文紹介: The Surprising Effectiveness of PPO in Cooperative Multi-Agent Games
 
LoRaWANスマート距離検出センサー DS20L カタログ LiDARデバイス
LoRaWANスマート距離検出センサー  DS20L  カタログ  LiDARデバイスLoRaWANスマート距離検出センサー  DS20L  カタログ  LiDARデバイス
LoRaWANスマート距離検出センサー DS20L カタログ LiDARデバイス
 
Amazon SES を勉強してみる その32024/04/26の勉強会で発表されたものです。
Amazon SES を勉強してみる その32024/04/26の勉強会で発表されたものです。Amazon SES を勉強してみる その32024/04/26の勉強会で発表されたものです。
Amazon SES を勉強してみる その32024/04/26の勉強会で発表されたものです。
 
知識ゼロの営業マンでもできた!超速で初心者を脱する、悪魔的学習ステップ3選.pptx
知識ゼロの営業マンでもできた!超速で初心者を脱する、悪魔的学習ステップ3選.pptx知識ゼロの営業マンでもできた!超速で初心者を脱する、悪魔的学習ステップ3選.pptx
知識ゼロの営業マンでもできた!超速で初心者を脱する、悪魔的学習ステップ3選.pptx
 
Amazon SES を勉強してみる その22024/04/26の勉強会で発表されたものです。
Amazon SES を勉強してみる その22024/04/26の勉強会で発表されたものです。Amazon SES を勉強してみる その22024/04/26の勉強会で発表されたものです。
Amazon SES を勉強してみる その22024/04/26の勉強会で発表されたものです。
 
論文紹介:Video-GroundingDINO: Towards Open-Vocabulary Spatio-Temporal Video Groun...
論文紹介:Video-GroundingDINO: Towards Open-Vocabulary Spatio-Temporal Video Groun...論文紹介:Video-GroundingDINO: Towards Open-Vocabulary Spatio-Temporal Video Groun...
論文紹介:Video-GroundingDINO: Towards Open-Vocabulary Spatio-Temporal Video Groun...
 
論文紹介:Selective Structured State-Spaces for Long-Form Video Understanding
論文紹介:Selective Structured State-Spaces for Long-Form Video Understanding論文紹介:Selective Structured State-Spaces for Long-Form Video Understanding
論文紹介:Selective Structured State-Spaces for Long-Form Video Understanding
 
Utilizing Ballerina for Cloud Native Integrations
Utilizing Ballerina for Cloud Native IntegrationsUtilizing Ballerina for Cloud Native Integrations
Utilizing Ballerina for Cloud Native Integrations
 
LoRaWAN スマート距離検出デバイスDS20L日本語マニュアル
LoRaWAN スマート距離検出デバイスDS20L日本語マニュアルLoRaWAN スマート距離検出デバイスDS20L日本語マニュアル
LoRaWAN スマート距離検出デバイスDS20L日本語マニュアル
 

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

Notas do Editor

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