Upgrade to Pro — share decks privately, control downloads, hide ads and more …

RailsConf 2013: Your First Rails Pull Request

RailsConf 2013: Your First Rails Pull Request

You have been doing this Rails thing for a while and you're starting to feel like it's time to give back. Great! Now what?
In this session we'll walk through the technical aspects of getting started with contributing back to Rails as well as the non-technical tips, tricks, and considerations to keep in mind along the way.

Mark McSpadden

May 01, 2013
Tweet

More Decks by Mark McSpadden

Other Decks in Programming

Transcript

  1. How to Submit a Pull Request to Rails Without Everyone

    Laughing at You Wednesday, May 1, 13
  2. How to Submit a Pull Request to Rails Without Being

    a Complete Jerk Wednesday, May 1, 13
  3. DISCLAIMER: I AM NOT ON THE CORE TEAM I AM

    NOT ON THE ISSUES TEAM Wednesday, May 1, 13
  4. The Rails Org Chart Core Team Issues Team Support Gem

    Committers Me You You2 Wednesday, May 1, 13
  5. Draft: The Rails Org Chart Core Team Issues Team Support

    Gem Committers Me You You dhh yehuda tenderlove Wednesday, May 1, 13
  6. Merged Pull Requests 74% < 24 hours to merge Merged

    Pull Requests Wednesday, May 1, 13
  7. Merged Pull Requests 90% < 9 days to merge Merged

    Pull Requests Wednesday, May 1, 13
  8. Merged Pull Requests 58% < 24 hours to reject Merged

    Pull Requests Rejected Pull Requests Wednesday, May 1, 13
  9. Merged Pull Requests 73% < 7 days to reject Merged

    Pull Requests Rejected Pull Requests Wednesday, May 1, 13
  10. Rails Issues Myth #1 There are lots of trivial Rails

    issues just waiting for you to swoop in a fix during your pomodoro break Wednesday, May 1, 13
  11. Rails Issues Myth #2 By commenting on an Issue, people

    will mistake you for someone important Wednesday, May 1, 13
  12. Ask yourself... • Does this match the style of the

    code around it? • Could this hurt performance? (Have a benchmarked it?) • What are the future maintenance implications of my changes? • Does this make Rails better? Wednesday, May 1, 13
  13. The Summary • Explain yourself and show your smarts &

    research • Link to any performance benchmarks, etc (gists) • cc people (git blame) Wednesday, May 1, 13