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
780
How to get your pull requests merged
Yasuo Honda
May 30, 2019
Tweet
Share
More Decks by Yasuo Honda
See All by Yasuo Honda
extensionとschema
yahonda
1
220
NOT VALIDな検査制約 / check constraint that is not valid
yahonda
1
160
今、始める、第一歩。 / Your first step
yahonda
3
1.1k
RailsのPull requestsのレビューの時に私が考えていること
yahonda
9
6.1k
pg_stat_statementsで inの数が違うSQLをまとめて ほしい
yahonda
0
230
遅延可能な一意性制約
yahonda
0
920
あなたとRails
yahonda
4
990
PostgreSQL 15とRailsと
yahonda
0
570
PostgreSQLとRuby on Rails
yahonda
0
250
Other Decks in Technology
See All in Technology
Go製のマイグレーションツールの git-schemalex の紹介と運用方法
shinnosuke_kishida
1
410
IAMのマニアックな話 2025 ~40分バージョン ~
nrinetcom
PRO
8
930
大規模サービスにおける カスケード障害
takumiogawa
2
260
Tirez profit de Messenger pour améliorer votre architecture
tucksaun
1
150
LINE Notify互換のボットを作った話
kenichirokimura
0
180
Engineering Managementのグローバルトレンド #emoasis / Engineering Management Global Trend
kyonmm
PRO
6
990
[CATS]Amazon Bedrock GenUハンズオン座学資料 #2 GenU環境でRAGを体験してみよう
tsukuboshi
0
150
DevOps文化を育むQA 〜カルチャーバブルを生み出す戦略〜 / 20250317 Atsushi Funahashi
shift_evolve
1
110
Riverpod & Riverpod Generatorを利用して状態管理部分の処理を書き換えてみる簡単な事例紹介
fumiyasac0921
0
110
20250328_RubyKaigiで出会い鯛_____RubyKaigiから始まったはじめてのOSSコントリビュート.pdf
mterada1228
0
130
パスキー導入の課題と ベストプラクティス、今後の展望
ritou
7
1.2k
SpannerとAurora DSQLの同時実行制御の違いに想いを馳せる
masakikato5
0
570
Featured
See All Featured
実際に使うSQLの書き方 徹底解説 / pgcon21j-tutorial
soudai
177
52k
Building a Modern Day E-commerce SEO Strategy
aleyda
39
7.2k
Unsuck your backbone
ammeep
670
57k
Documentation Writing (for coders)
carmenintech
69
4.7k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
32
2.2k
BBQ
matthewcrist
88
9.5k
No one is an island. Learnings from fostering a developers community.
thoeni
21
3.2k
Helping Users Find Their Own Way: Creating Modern Search Experiences
danielanewman
29
2.5k
How STYLIGHT went responsive
nonsquared
99
5.4k
Become a Pro
speakerdeck
PRO
27
5.2k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
251
21k
Rails Girls Zürich Keynote
gr2m
94
13k
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