Jetpack/Contribution

From MozillaWiki
Jump to navigation Jump to search

Overview

  • Changes should follow the design guidelines for Jetpack
  • All changes must be accompanied by tests
  • In order to land, patches must have review from a core Jetpack developer
  • Patches should have additional API review when needed
  • Patches should have additional review from a Mozilla platform domain-expert when needed

If you have questions, ask in #jetpack on IRC or on the Jetpack mailing list.

How to Make Code Contributions

If you have code that you'd like to contribute the Jetpack project, follow these steps:

  1. Look for your issue in the bugs already filed
  2. If no bug exists, file one here
  3. Make your changes, per the Overview
  4. Write a test (intro, API)
  5. Attach your code changes and tests in a combined patch to the bug
  6. Ask for code review by setting one or more "review" flags on your attachment to "?" and entering the email of the appropriate developer (noted in Reviewers below)

Reviewers

  • All changes must be reviewed by someone on the Jetpack review crew:
    • Alex (apoirot@mozilla.com, in bugzilla: poirot.alex@gmail.com)
    • Brian (warner@mozilla.com, in bugzilla: warner-bugzilla@lothar.com)
    • Dave Mason (dcm@mozilla.com)
    • Dave Townsend (mossop@mozilla.com, in bugzilla: dtownsend+bugmail@oxymoronical.com)
    • Dietrich (dietrich@mozilla.com)
    • Drew (adw@mozilla.com)
    • Irakli (gozala@mozilla.com, in bugzilla: rFobic@gmail.com)
    • Mark (mhammond@mozilla.com, in bugzilla: mhammond@skippinet.com.au)
    • Myk (myk@mozilla.org)
    • Will (wbamberg@mozilla.com)
  • For review of Mozilla platform usage and best practices, ask Dietrich, Drew, or Dave Townsend to find the domain expert.
  • For API and developer ergonomics review, ask Irakli.