Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
How to get your pull requests merged
Search
Yasuo Honda
May 30, 2019
Technology
9
840
How to get your pull requests merged
Yasuo Honda
May 30, 2019
Tweet
Share
More Decks by Yasuo Honda
See All by Yasuo Honda
Railsの話をしよう
yahonda
0
180
RailsのPostgreSQL 18対応
yahonda
0
2.3k
Contributing to Rails? Start with the Gems You Already Use
yahonda
2
180
PostgreSQL 18 cancel request key長の変更とRailsへの関連
yahonda
0
230
extensionとschema
yahonda
1
280
NOT VALIDな検査制約 / check constraint that is not valid
yahonda
1
210
今、始める、第一歩。 / Your first step
yahonda
3
1.3k
RailsのPull requestsのレビューの時に私が考えていること
yahonda
11
7.4k
pg_stat_statementsで inの数が違うSQLをまとめて ほしい
yahonda
0
260
Other Decks in Technology
See All in Technology
ヘンリー会社紹介資料(エンジニア向け) / company deck for engineer
henryofficial
0
400
AI時代の開発を加速する組織づくり - ブログでは書けなかったリアル
hiro8ma
2
330
プレイドのユニークな技術とインターンのリアル
plaidtech
PRO
1
460
AIエージェントによる業務効率化への飽くなき挑戦-AWS上の実開発事例から学んだ効果、現実そしてギャップ-
nasuvitz
5
1.4k
Kubernetes self-healing of your workload
hwchiu
0
580
OTEPsで知るOpenTelemetryの未来 / Observability Conference Tokyo 2025
arthur1
0
290
ハノーファーメッセ2025で見た生成AI活用ユースケース.pdf
hamadakoji
1
490
様々なファイルシステム
sat
PRO
0
260
Behind Postgres 18: The People, the Code, & the Invisible Work | Claire Giordano | PGConfEU 2025
clairegiordano
0
150
SRE × マネジメントレイヤーが挑戦した組織・会社のオブザーバビリティ改革 ― ビジネス価値と信頼性を両立するリアルな挑戦
coconala_engineer
0
280
クラウドとリアルの融合により、製造業はどう変わるのか?〜クラスメソッドの製造業への取組と共に〜
hamadakoji
0
450
猫でもわかるAmazon Q Developer CLI 解体新書
kentapapa
1
120
Featured
See All Featured
Making Projects Easy
brettharned
120
6.4k
GraphQLとの向き合い方2022年版
quramy
49
14k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
285
14k
The Language of Interfaces
destraynor
162
25k
Testing 201, or: Great Expectations
jmmastey
45
7.7k
It's Worth the Effort
3n
187
28k
No one is an island. Learnings from fostering a developers community.
thoeni
21
3.5k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
253
22k
Rebuilding a faster, lazier Slack
samanthasiow
84
9.2k
The Art of Delivering Value - GDevCon NA Keynote
reverentgeek
16
1.7k
KATA
mclloyd
PRO
32
15k
For a Future-Friendly Web
brad_frost
180
10k
Transcript
How to get your pull requests merged ۜ࠲Rails#9 2019/05/30 @yahonda
ࣗݾհͱ2ͭͷࢹ • Rails contributor • Maintainer of Active Record Oracle
enhanced adapter • Pull requestΛૹΔཱͱड͚औΔཱ
͢͜ͱɺ͞ͳ͍͜ͱ • ͢͜ͱ • RailsϑϨʔϜϫʔΫͷ։ൃମ੍ • Railsʹpull requestΛૹΔͱ͖ʹ͢Δ͜ͱɺ͠ͳ͍͜ͱ • ͞ͳ͍͜ͱ
• ίʔυ • https://railsguides.jp/contributing_to_ruby_on_rails.html
͋ͳͨͳͥRailsʹpull requestΛ͓͘Δͷ͔
͋ͳͨͷ՝Λղܾ͢ΔͨΊ
ͦͷͨΊʹඞཁͳ͜ͱ
ͦͷpull request͕Railsʹऔ Γࠐ·ΕΔ͖ཧ༝Λίϛο λʔʹೲಘͯ͠Β͏͜ͱ
RailsϑϨʔϜϫʔΫ։ൃମ੍ • ϦϦʔεεέδϡʔϧ • ίϛϡχςΟ • Pull requestΛϚʔδͰ͖Δͷ୭͔ • ݴޠ
• ࣌ࠩ
͋ͳͨͷRailsΞϓϦέʔγϣ ϯ։ൃମ੍ͱಉ͡ͱ͜Ζ͋ Δ͠ɺҧ͏ͱ͜Ζ͋Δ
ϦϦʔεεέδϡʔϧ • RailsͷϦϦʔεαΠΫϧ • Major, Minor: ୯ҐɺTeeny: ݄୯Ґ • Ծʹࠓɺpull
requestϚʔδ͞Εͯೖͬͨ৽ػೳ͕Ϧ Ϧʔε͞ΕΔͷRails 6.1(͓ͦΒ͘2020Ҏ߱) • ͋ͳͨͷRailsΞϓϦέʔγϣϯͷϦϦʔεαΠΫϧ ɺ͓ͦΒͦ͘ΕΑΓ͍ͣ
ίϛϡχςΟ • DHH / Rails core / Rails committer •
Rails contributors • ಉ͡ձࣾ/৫ͰຖҰॹʹ͍ΔΘ͚Ͱͳ͍ • ίϛϡχςΟϝϯόʔ͕RailsʹٻΊΔͷͦΕͧΕҟ ͳΔ • ͋ͳ͕ͨRailsʹٻΊΔͷΛΘ͔ͬͯΒ͏ඞཁ͕͋Δ
Pull requestΛϚʔδ͢Δͷ • ࠷େ18ਓͷRailsίϛολʔ • ΞΫςΟϒͳίϛολʔͬͱগͳ͍ • 700Ҏ্ͷopen pull requests
• 39 pull requests / ίϛολʔ • ͋ͳͨͷRailsΞϓϦέʔγϣϯͷϨϏϡʔମ੍ͱൺͯΈ ·͠ΐ͏
ݴޠ • Ruby • ίʔυ • ӳޠ • ίϛοτϝοηʔδ •
Pull requestͰͷΓͱΓ
࣌ࠩ • Rails ίϛολʔੈքதʹ͍Δ • ͋ͳͨͷ৸͍ͯΔؒʹpull request͕ϨϏϡʔ ͞ΕΔ͔͠Εͳ͍
Railsʹpull requestΛૹΔ ࣌ɺͦͯ͠ૹͬͨޙʹͳʹΛ ͢Δ͔ɺ͠ͳ͍͔
Pull requestΦʔϓϯ͢Δ࣌ʹ • Pull requestΛϚʔδ͢Δཧ༝Λઆ໌͍ͯ͠Δ͔ • ͋ͳͨͷ՝͔ͩΒཧ༝આ໌Ͱ͖Δͣ • Ͱ͖Δ͜ͱΛΔɺͰ͖Δ͜ͱΔ •
ʮ͍ͭͷ͔ૹΔʯͬͯ͜ͳ͍͔͠Εͳ͍ • ʮ͜͏ͩͬͨΒ͍͍ͷʹʯͱ͍͏ίϝϯτΛॻ͘ͳ Βɺগ͠Ͱͦͷʮ͜͏ʯΛίʔυʹ͢Δ
Pull requestΛΦʔϓϯͨ͠ޙ • ͋ͤΒͳ͍ͰԠΛͭ • Ϛʔδͯ͠Β͏ͨΊʹඞཁͳใΛॻ͖ ͍ͬͯΕɺޙ͢Δ͜ͱͳ͍ͣ
ϨϏϡʔ࣭͕དྷͨΒ • Pull requestͰίʔυ͔ӳޠͰճ͢Δ • ڵຯΛ࣋ͨΕ͍ͯΔؒʹճ͢Δ • ϨϏϡʔ࣭Λແࢹ͠ͳ͍
ෳਓ͔Βίϝϯτ͕དྷͨΒ • ίϛολʔʹೲಘͯ͠Β͏͜ͱʹྗ͢Δ • Ϛʔδ͢Δݖݶͷͳ͍ਓΛઆಘͯ͠ํ͕ ͳ͍ɺͱ͍͏ݱ࣮͋Δ • ͘ͳΕͳΔ΄Ͳऩर͕͚ͭʹ͍͘
ϨϏϡʔ͕Θ͔Βͳ͔ͬͨΒ • Θ͔͍ͬͯΔ෦ΛαϚϦʔͯ͠ฦࣄ͢Δ • Βͳ͔ͬͨ͜ͱʹରͯ͠Sorryͱ͍Θͳ͍ • Βͳ͔ͬͨ͜ͱΛڭ͑ͯ͘Εͨ͜ͱʹײँ Λ͠ɺͦΕΛ;·͑ͨίʔυมߋΛ͢Δ
ϨϏϡʔʹࢍͰ͖ͳ͔ͬͨΒ • ࢍͰ͖ͳ͍ཧ༝Λॻ͘ • ʮϨϏϡʔ͋Γ͕ͱ͏ɺͰࢲ͜Ε͕ ΈʯͰΑ͍ • ୯७ʹແࢹ͠ͳ͍
୭͔ΒԿͷԠͳ͔ͬͨΒ • ΈΜͳ͕͏Ձ͕͋Δͷͩͱpull request ୯ಠͰઆ໌͍ͯ͠Δ͔ɺ͏͍ͪͲಡΜͰΈ ·͠ΐ͏ • ͋ͳͨͷϢʔεέʔεઆ໌͞Ε͍ͯ·͔͢ • ڵຯΛҾ͘pull
requestͰ͋ΕɺԿ͔͠Βͷ Ԡ͕͋Δͣ
Pull requestͷϝϯςφϯε • ίϯϑϦΫτΛղফ͓ͯ͘͠ • CIgreenʹ͓ͯ͘͠
Pull request͕ෆཁʹͳͬͨΒ • Ϋϩʔζ͢Δ • ʮࢲ͏ཁΒͳ͍͚Ͳɺଞͷ୭͔͕ඞཁ͔ ͠Εͳ͍ʯͱ͍͏ͷߟ͑ͳͯ͘Α͍
Pull requestΛΦʔϓϯͨ͠ޙ ʹΒͳ͍͜ͱ • ping͠ͳ͍ / any updates? ͱ͔͍Θͳ͍ •
ʮϚʔδ͖͢ཧ༝ʯΛҰͭઆ໌͍ͯ͠ͳ ͍͔Β
Ϛʔδ͞ΕͣΫϩʔζ͞ΕͨΒ • Ϋϩʔζ͞Εͨissuepull requestʹίϝϯτ ͠ͳ͍ • ʮࣺͯࢺʯ͔͔ͳ͍
Ϋϩʔζ͞Εͨpull requestΛ ࠶νϟϨϯδͨ͘͠ͳͬͨΒ • ͍ͬͨΜΫϩʔζ͞Εͨpull requestͱಉ͡ࡐ Λ෮׆ͤ͞ΔͷࠔͰ͋Δ • ͕࣌ؒܦͭͷΛͭ •
ൣғΛখ͘͢͞ΔɺΞϓϩʔνΛม͑Δ • அ͕มΘΔ͔͠Εͳ͍ɺલఏ͕มΘΔ͔ ͠Εͳ͍ɺϝϯςφʔ͕ସΘΔ͔͠Εͳ͍
Pull request͕Ϛʔδ͞ΕͨΒ • https://contributors.rubyonrails.org • https://y-yagi.hatenablog.com • ࣍ͷRails͕ϦϦʔε͞ΕΔͷΛͪ·͠ΐ͏ • ͋ͳͨͷcommitͷੵΈॏͶͰɺ͋ͳͨRails
ίϛϡχςΟʹΒΕ͍ͯ͘Α͏ʹͳΓ·͢
OSSύονձͲ͏ͧ ࣍ճ6݄6