Closed
Bug 1569140
Opened 5 years ago
Closed 5 years ago
Fix rustfmt warnings in geckodriver and webdriver
Categories
(Testing :: geckodriver, task)
Tracking
(firefox70 fixed)
RESOLVED
FIXED
mozilla70
| Tracking | Status | |
|---|---|---|
| firefox70 | --- | fixed |
People
(Reporter: ato, Assigned: ato)
Details
Attachments
(1 file)
Bug 1568889 enabled the rustfmt linter check in CI,
and while this didn’t cause any errors,
whimboo suggested we should go ahead and fix the warnings.
|
Assignee | |
Comment 1•5 years ago
|
||
Bug 1568889 enabled the rustfmt lint checks in continuous integration,
and while they did not cause any errors, this is a follow-up patch
to fix the non-fatal warnings.
|
|
Assignee | |
Comment 2•5 years ago
|
||
After this change:
% rustfmt --check src/*.rs
(No output, which means good.)
I will leave this patch up for consideration.
As I mentioned in https://phabricator.services.mozilla.com/D39368#1185605
we may not want to apply this unconditionally.
Pushed by atolfsen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/610a3463ba30 webdriver: lint; r=webdriver-reviewers,automatedtester
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
|
||
Updated•5 years ago
|
Assignee: nobody → ato
You need to log in
before you can comment on or make changes to this bug.
Description
•