-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[js] Add service example #1848
[js] Add service example #1848
Conversation
PR Reviewer Guide 🔍
|
CI Failure Feedback 🧐(Checks updated until commit cc00794)
✨ CI feedback usage guide:The CI feedback tool (
In addition to being automatically triggered, the tool can also be invoked manually by commenting on a PR:
where Configuration options
See more information about the |
PR Code Suggestions ✨
|
✅ Deploy Preview for selenium-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @pujagani !
User description
Thanks for contributing to the Selenium site and documentation!
A PR well described will help maintainers to review and merge it quickly
Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, and help reviewers by making them as simple and short as possible.
Description
Add code examples, especially since the Selenium Manager links to a code snippet to use service for certain architectures that are not supported by Selenium Manager.
Motivation and Context
Types of changes
Checklist
PR Type
Tests
Description
service.spec.js
underexamples/javascript/test/drivers/
.selenium-webdriver
library to build and manage the Chrome driver instances.Changes walkthrough 📝
service.spec.js
Add Chrome service example tests in JavaScript
examples/javascript/test/drivers/service.spec.js
location, and setting port.
selenium-webdriver
library for browser automation.