WebExtensions/Reviews: Difference between revisions
Jump to navigation
Jump to search
(clarified peer review is always required) |
m (→Other reviewers: fixed typo (missing 'the')) |
||
| (One intermediate revision by one other user not shown) | |||
| Line 7: | Line 7: | ||
== Other reviewers == | == Other reviewers == | ||
Other members of the team are more than happy to review patches and will pass them up to the peers before landing. So if you got a bug in one of following areas, consider contacting one of these before hand. | Other members of the team are more than happy to review patches and will pass them up to the peers before landing. So if you got a bug in one of the following areas, consider contacting one of these before hand. | ||
* Bob Silverberg (bsilverberg on IRC) | * Bob Silverberg (bsilverberg on IRC) | ||
| Line 13: | Line 13: | ||
* Luca Greco (rpl on IRC) | * Luca Greco (rpl on IRC) | ||
** webNavigation, devtools, tabs, content-scripts, addon debugging tools | ** webNavigation, devtools, tabs, content-scripts, addon debugging tools | ||
* Tomislav Jovanovic (zombie on IRC) | * Tomislav Jovanovic (zombie on IRC) | ||
** downloads, runtime, browserAction, pageAction, contextMenus, tabs, windows, webNavigation, webRequest, content-scripts | ** downloads, runtime, browserAction, pageAction, contextMenus, tabs, windows, webNavigation, webRequest, content-scripts | ||
Latest revision as of 16:05, 21 August 2017
You've got a patch in WebExtensions that needs a review? Great, here's how to go about it.
Peers
The current (May 2017) peers for WebExtensions are Andrew Swan (aswan on IRC), Kris Maglione (John-Galt on IRC) and Shane Caraveo (mixedpuppy on IRC). These are also the peers that will show up in bugzilla as suggested reviewers.
Other reviewers
Other members of the team are more than happy to review patches and will pass them up to the peers before landing. So if you got a bug in one of the following areas, consider contacting one of these before hand.
- Bob Silverberg (bsilverberg on IRC)
- alarms, bookmarks, browsingData, history, management, notifications, privacy, sessions
- Luca Greco (rpl on IRC)
- webNavigation, devtools, tabs, content-scripts, addon debugging tools
- Tomislav Jovanovic (zombie on IRC)
- downloads, runtime, browserAction, pageAction, contextMenus, tabs, windows, webNavigation, webRequest, content-scripts