Gaia/UX Branch/Maintenance: Difference between revisions

Jump to navigation Jump to search
no edit summary
No edit summary
Line 1: Line 1:
== Commits Merged into UX Branch ==
Reverse-chronological.
* 2012-12-04
** bug-802782 <https://github.com/gordonbrander/gaia/commit/b400577ed1005cba4f954fa8251336f2e90fadf3> Update camera icons
** bug-814688 <https://github.com/gordonbrander/gaia/commit/4e4c25b36eed2138ec5e6076d356e0ba653fda3e> Update calendar icons
** bug-806963 <https://github.com/gordonbrander/gaia/commit/6e0a20e56fb25a11abb8ea3e954f8fcf0f91bd97> Bug806963_Updates_Incorrect_Design
* 2012-12-03
** bug-817117 <https://github.com/gordonbrander/gaia/commit/2a3b402d15bcd6c39fbd6f6b41e1f22d8bd36162>
** bug-813605 <https://github.com/gordonbrander/gaia/pull/35/files> Notification usability/font/sound
** bug-813091 <https://github.com/gordonbrander/gaia/pull/38> dialer iconography
** bug-811640 <https://github.com/gordonbrander/gaia/commit/d03692c11d5642b968655fb6614d9bb4d019990a> [KEYBOARD] Long press on 0 or p or }, the keyboard would shift left a little bit
** bug-802747 <https://github.com/gordonbrander/gaia/pull/36> gallery icon updates. Verified file names.
** bug-813275 <https://github.com/gordonbrander/gaia/pull/37> Browser Font Tweaks
* 2012-11-30
** bug-814753 <https://github.com/gordonbrander/gaia/commit/7eae08c77986d97c6a38c79bf71646ebd6c225aa> <https://github.com/gordonbrander/gaia/commit/6df8fa7b7cd66181d081a0211eea724f808d34fb> [status bar] Update Status bar Icons
** bug-802935, bug-806972, bug-806974 - <https://github.com/gordonbrander/gaia/pull/30> <https://github.com/gordonbrander/gaia/commit/6854fc1720c09b33fecda6629975936b6cbc0688>
** bug-813140 - <https://github.com/gordonbrander/gaia/pull/31>  <https://github.com/gordonbrander/gaia/commit/e9f8085a01fd40c3bf6f68ba534052992910f584> Contacts iconography fix. I had to patch this one in because of commit history.
* 2012-11-29
** bug-802648 - <https://github.com/gordonbrander/gaia/pull/23> update browser icons
** bug-814390 - <https://github.com/gordonbrander/gaia/pull/26> fix purple highlight for settings
* 2012-11-27
** bug-813703 - <https://github.com/gordonbrander/gaia/pull/17> FM Radio font changes
** REVERTED bug-813275 - <https://github.com/gordonbrander/gaia/pull/18> Browser font changes
* 2012-11-22
** bug-813603 - <https://github.com/gordonbrander/gaia/commit/8465ef3e65870f4a2ea79bf162905cedb115c6d5> Shared header font size tweaks.
** bug-813560 - <https://github.com/gordonbrander/gaia/commit/0dd64b35281d893ae22eb2a092decad7295957a4> Bug813560_Settings_Font_Tweaks
** bug-808676 - <https://github.com/gordonbrander/gaia/commit/8bfb812be56e19f79062264b8659bdba9e1da103>
* 2012-11-16
** bug-805129 - Browser toolbar fix https://github.com/gordonbrander/gaia/commit/6ff5ae8111812f4715dd7ebc53df04f123402c9b
* 2012-11-13
** bug-809611 - Keyboard highlights: <https://github.com/gordonbrander/gaia/commit/5f41767e48c3222188546dfb5a6c2b925bd71848>
* 2012-11-09
** bug-809294: <https://github.com/gordonbrander/gaia/commit/dc21da3db0c4751af1612b7301d25194f081ab2c> FM Radio Dial garbage commit (fixed in master)
* 2012-11-06
** bug-809294: FM Radio Dial (fixed in master) <https://github.com/gordonbrander/gaia/commit/bc5ff642fa2af28a58a9cfa64e6f6b3886fb61d0>
* 2012-11-05
** bug-808761: <https://github.com/gordonbrander/gaia/commit/19259084ee4d340ffac58d74f180e0d505e406dd>
== Pull Requests ==
== Pull Requests ==


Line 61: Line 11:
* Review pull requests 2-3 times a week
* Review pull requests 2-3 times a week
   * Open bug in Bugzilla
   * Open bug in Bugzilla
   * Tag bug with uxbranch (if it doesn't have the tag already)
   * Add Status Whiteboard "uxbranch" (if it doesn't have the tag already)
   * Review patch, understand risk-level of patch
   * Review patch, understand risk-level of patch
   * Apply patch locally and test on-device
   * Apply patch locally and test on-device
   * If all is OK, merge pull request.
   * If all is OK, merge pull request.
   * Only one commit per fix! Make sure commit has bug # and details. If necessary, apply as patch and fix commit message.
   * Only one commit per fix! Make sure commit has bug # and details. If necessary, apply as patch and fix commit message.
  * Add Status Whiteboard "landed in uxbranch"
* Create a pull request for qa-verified bugs every week.
* Create a pull request for qa-verified bugs every week.
   * Cherry pick [commits verified by QA and merged into UX, but not yet in master][prcommits].
   * Cherry pick commits verified by QA and merged into UX, but not yet in master. Prefer cherry-picking the original commit instead of the merge commit.
   * Issue pull request.
   * Issue pull request.
  * Include links to bugs in pull request message.


Good news: cherry pick should fail noisily if it can't merge properly.
Good news: cherry pick should fail noisily if it can't merge properly.
Note: you can get the original commit for a merge by clicking on the right-hand parent commit.
Note: you can get the original commit for a merge by clicking on the right-hand parent commit.


== Patching ==
== Patching ==


To keep commit history clean, sometimes using git patch instead of pull requests.
To keep commit history clean, sometimes it makes sense to use git patch instead of merging a pull request.


When patching, use this format:
When patching, use this format:
canmove, Confirmed users
227

edits

Navigation menu