WebDriver/Meetings/2018-08-06: Difference between revisions
Jump to navigation
Jump to search
(David's status) |
(Henrik's updates) |
||
| Line 1: | Line 1: | ||
== Agenda == | == Agenda == | ||
* Selection of right Review Tool while being in a transition from MozReview to Phabricator | |||
** Should we continue using MozReview until EOL? Or immediately switch to Phabricator? Or use the old Splinter review? | |||
** MozReview will still allow creating new patches until August 26th | |||
** What's missing in Phabricator? | |||
*** Patch series aren't a supported feature yet, but extensions fill the gap for both hg and git | |||
**** https://bitbucket.org/kmaglione/hgext/src/default/phabricator.py | |||
**** https://github.com/mystor/phlay | |||
** Splinter is bad in many ways | |||
*** Updates of the patches remove former review status which require additional checks in bug comments to see details of a former review | |||
*** Issues as created during the review do not exist. It's all about comments ... | |||
*** ... which then even don't have to be replied to in Splinter, which makes following conversations very hard | |||
== Minutes == | == Minutes == | ||
| Line 11: | Line 22: | ||
* Henrik | * Henrik | ||
** '''[wpt] Increased startup/shutdown timeout for Firefox to catch crashes caused by shutdown hangs (Bug 1456746)''' | |||
** '''[wdspec] Added remaining prompt behavior tests for existent user prompt tests (Bug 1479850)''' | |||
** '''[wdspec] About to move shared methods into __init__.py file for each command (Bug 1480337)''' | |||
*** '''[wdspec] I will start working on Element Send Keys webdriver compatibility support soon''' | |||
*** '''[geckodriver] Added geckodriver release mapping to Firefox and Selenium: https://github.com/mozilla/geckodriver/wiki/Geckodriver-Releases''' | |||
** '''[geckodriver] Continued converting code to Serde. Close to be able to build geckodriver (Bug 1396821)''' | |||
** [wdspec] Added tests for no browsing context (Bug 1478799) | |||
** [wdspec] Removed unnecessary createFile fixture for file upload tests (Bug 1479677) | |||
** [marionette] Fix for Execute Script if script body contains a comment in the last line (Bug 1337899) | |||
* David | * David | ||
Revision as of 09:44, 6 August 2018
Agenda
- Selection of right Review Tool while being in a transition from MozReview to Phabricator
- Should we continue using MozReview until EOL? Or immediately switch to Phabricator? Or use the old Splinter review?
- MozReview will still allow creating new patches until August 26th
- What's missing in Phabricator?
- Patch series aren't a supported feature yet, but extensions fill the gap for both hg and git
- Splinter is bad in many ways
- Updates of the patches remove former review status which require additional checks in bug comments to see details of a former review
- Issues as created during the review do not exist. It's all about comments ...
- ... which then even don't have to be replied to in Splinter, which makes following conversations very hard
Minutes
Status updates
(Spoken status updates in bold.)
- ato
- Henrik
- [wpt] Increased startup/shutdown timeout for Firefox to catch crashes caused by shutdown hangs (Bug 1456746)
- [wdspec] Added remaining prompt behavior tests for existent user prompt tests (Bug 1479850)
- [wdspec] About to move shared methods into __init__.py file for each command (Bug 1480337)
- [wdspec] I will start working on Element Send Keys webdriver compatibility support soon
- [geckodriver] Added geckodriver release mapping to Firefox and Selenium: https://github.com/mozilla/geckodriver/wiki/Geckodriver-Releases
- [geckodriver] Continued converting code to Serde. Close to be able to build geckodriver (Bug 1396821)
- [wdspec] Added tests for no browsing context (Bug 1478799)
- [wdspec] Removed unnecessary createFile fixture for file upload tests (Bug 1479677)
- [marionette] Fix for Execute Script if script body contains a comment in the last line (Bug 1337899)
- David
- OKRs for H2
PTO/travel (🏖)
- Monday 27 August is a bank holiday in the UK
- ato away Friday 10 August