Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'How to build driver with phantomjs-prebuilt' #2086

Merged
merged 3 commits into from
May 21, 2016
Merged

Add 'How to build driver with phantomjs-prebuilt' #2086

merged 3 commits into from
May 21, 2016

Conversation

vtange
Copy link
Contributor

@vtange vtange commented May 11, 2016

Added Instructions for something like this in the comments above "Find Executable" function of PhantomJS
In response to #2069

Added Instructions for something like this in the comments above "Find Executable" function of PhantomJS
In response to #2069
@jleyba
Copy link
Contributor

jleyba commented May 19, 2016

You've updated the docs on a private function. If you want this notice to be included in the generated API docs, you should add a fileoverview comment to the top of the file. For example, see selenium-webdriver/chrome.js (here are the public docs generated extracted from that file).

@vtange
Copy link
Contributor Author

vtange commented May 19, 2016

OK. Moved the docs to the top of the file under @fileoverview. Ring me up if it still needs work

@@ -15,6 +15,28 @@
// specific language governing permissions and limitations
// under the License.

/**
* @fileoverview Defines a {@linkplain Driver WebDriver} client for the PhantomJS
* web browser. By default, it is expected that PhantomJS exists in your working directory.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default, it is expected that the PhantomJS executable can be located on your
[PATH](https://en.wikipedia.org/wiki/PATH_(variable))

@jleyba jleyba merged commit 1eb3d09 into SeleniumHQ:master May 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants