Closed Bug 1369086 Opened 7 years ago Closed 7 years ago

Create Testing / GeckoDriver component

Categories

(bugzilla.mozilla.org :: Administration, task)

Production
task
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: whimboo, Assigned: dylan)

References

Details

Description:

Proxy for using W3C WebDriver-compatible clients to interact with Gecko-based browsers.
This should be s/GeckoDriver/geckodriver/.
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.
(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.
Please look at existing components and how they are spelled.

Anyway, lets get feedback from David in what we should use.
Flags: needinfo?(dburns)
(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.
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
Then please go ahead with the lowercase version. I just want to have this component. Thanks.
Flags: needinfo?(dburns)
 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.