Closed
Bug 1369086
Opened 7 years ago
Closed 7 years ago
Create Testing / GeckoDriver component
Categories
(bugzilla.mozilla.org :: Administration, task)
Tracking
()
RESOLVED
FIXED
People
(Reporter: whimboo, Assigned: dylan)
References
Details
Description: Proxy for using W3C WebDriver-compatible clients to interact with Gecko-based browsers.
Comment 1•7 years ago
|
||
This should be s/GeckoDriver/geckodriver/.
Reporter | ||
Comment 2•7 years ago
|
||
Why that? All components start with a capital letter, and mostly have camel-case. Also regarding the camel-case topic there is the `WebDriver` spec.
Comment 3•7 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #2) > Why that? All components start with a capital letter, and mostly have > camel-case. Also regarding the camel-case topic there is the `WebDriver` > spec. geckodriver is not spelt in camel-case or with a capital G. It is spelt like a package name.
Reporter | ||
Comment 4•7 years ago
|
||
Please look at existing components and how they are spelled. Anyway, lets get feedback from David in what we should use.
Flags: needinfo?(dburns)
Comment 5•7 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #4) > Please look at existing components and how they are spelled. In that case, please don’t misinterpret the name of the program. Like all other Unix programs, it is written with lower casing. Even the version information and the naming in all the official documentation [1] agrees: > % geckodriver --version > geckodriver 0.16.1 > > The source code of this program is available at > https://github.com/mozilla/geckodriver. > > This program is subject to the terms of the Mozilla Public License 2.0. > You can obtain a copy of the license at https://mozilla.org/MPL/2.0/ [1] https://searchfox.org/mozilla-central/source/testing/geckodriver/README.md Also, it is simply not true that Bugzilla components must start with capital letters, as this list shows: https://bugzilla.mozilla.org/describecomponents.cgi?product=Testing Historically, the reason for WebDriver’s camel case is that it was a Java implementation, and the Java coding style uses camel casing for package names.
Assignee | ||
Comment 6•7 years ago
|
||
Unless :automatedtester disagrees, I'll create this with the lower case name. Note that most search operations are case insensitive (thanks, mysql), so you can still write enter_bug.cgi?component=GECKODRIVER&product=TeSTING if you like.
Assignee: nobody → dylan
Reporter | ||
Comment 7•7 years ago
|
||
Then please go ahead with the lowercase version. I just want to have this component. Thanks.
Flags: needinfo?(dburns)
Assignee | ||
Comment 8•7 years ago
|
||
The component geckodriver has been created.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•