Confirmed users
764
edits
Jetpack/Contribution: Difference between revisions
Jump to navigation
Jump to search
m
→How to Make Code Contributions: Update out-of-date link to unit-test API
(→Reviewers: update brian's bugzilla address) |
m (→How to Make Code Contributions: Update out-of-date link to unit-test API) |
||
| Line 18: | Line 18: | ||
# If no bug exists, [https://bugzilla.mozilla.org/enter_bug.cgi?product=Add-on%20SDK&component=general file one here] | # If no bug exists, [https://bugzilla.mozilla.org/enter_bug.cgi?product=Add-on%20SDK&component=general file one here] | ||
# Make your changes, per the [[Labs/Jetpack/Contribution#Overview|Overview]] | # Make your changes, per the [[Labs/Jetpack/Contribution#Overview|Overview]] | ||
# Write a test ([https://jetpack.mozillalabs.com/sdk/latest/docs/#guide/packaging intro], [https://jetpack.mozillalabs.com/sdk/latest/docs/#module/ | # Write a test ([https://jetpack.mozillalabs.com/sdk/latest/docs/#guide/packaging intro], [https://jetpack.mozillalabs.com/sdk/latest/docs/#module/api-utils/unit-test API]) | ||
# Attach your code changes and tests in a combined patch to the bug | # Attach your code changes and tests in a combined patch to the bug | ||
# 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 [[Labs/Jetpack/Contribution#Reviewers|Reviewers]] below) | # 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 [[Labs/Jetpack/Contribution#Reviewers|Reviewers]] below) | ||