Conversation
|
@mach6 @lukeis @titusfortner please review |
|
Currently build in Travis failed with: (https://travis-ci.org/SeleniumHQ/selenium/jobs/187081577#L1338) and (https://travis-ci.org/SeleniumHQ/selenium/jobs/187081595#L870) So, it's definitely not because of the typos fixing in this PR. |
cgoldberg
left a comment
There was a problem hiding this comment.
why remove double spaces from the license header in capabilities.js when the same text appears in nearly every source file (with the double spaces)? seems like an arbitrary change making things inconsistent.
|
I see your point @cgoldberg, agreed. Added back extra spaces in license for consistency. Please re-review and merge if ok |
|
👍 I don't have access to merge though |
|
Thanks for reviewing! |
Xin the preceding checkbox, I verify that I have signed the Contributor License Agreement