added skip-driver-in-path argument to selenium_manager.en.md#2254
added skip-driver-in-path argument to selenium_manager.en.md#2254boris779 wants to merge 4 commits into
Conversation
added documentation for the argument skip-driver-in-path
👷 Deploy request for selenium-dev pending review.Visit the deploys page to approve it
|
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
|
Thank you for the PR! Could you please create an issue ticket describing this line: "needed for Problems with snap geckodriver" |
Code Review by Qodo
Context used✅ Compliance rules (platform):
10 rules 1. Duplicate argument row
|
| |`--browser-version <BROWSER_VERSION>`|`browser-version = "BROWSER_VERSION"`|`SE_BROWSER_VERSION=BROWSER_VERSION`|Major browser version (e.g., `105`, `106`, etc. Also: `beta`, `dev`, `canary` -or `nightly`-, and `esr` -in Firefox- are accepted)| | ||
| |`--driver-version <DRIVER_VERSION>`|`driver-version = "DRIVER_VERSION"`|`SE_DRIVER_VERSION=DRIVER_VERSION`|Driver version (e.g., `106.0.5249.61, 0.31.0`, etc.)| | ||
| |`--browser-path <BROWSER_PATH>`|`browser-path = "BROWSER_PATH"`|`SE_BROWSER_PATH=BROWSER_PATH`|Browser path (absolute) for browser version detection (e.g., `/usr/bin/google-chrome`, `/Applications/Google Chrome.app/Contents/MacOS/Google Chrome`, `C:\Program Files\Google\Chrome\Application\chrome.exe`)| | ||
| |`--skip-driver-in-path`|`skip-driver-in-path = true`|`SE_SKIP_DRIVER_IN_PATH=true`|Not using drivers found in the PATH| |
There was a problem hiding this comment.
1. Duplicate argument row 🐞 Bug ≡ Correctness
selenium_manager.en.md now lists --skip-driver-in-path twice in the arguments table (same CLI/config/env values), which creates conflicting/duplicated documentation. Readers will see the option twice and future edits may accidentally update only one entry.
Agent Prompt
### Issue description
The Selenium Manager arguments table contains two identical entries for `--skip-driver-in-path` after this PR, which results in duplicate documentation.
### Issue Context
The PR adds a new row for `--skip-driver-in-path` near the top of the table, but the same argument is already documented later in the same table.
### Fix Focus Areas
- website_and_docs/content/documentation/selenium_manager.en.md[112-132]
### Suggested change
Remove one of the two `--skip-driver-in-path` rows (preferably the newly-added duplicate), and keep a single canonical entry with consistent formatting (e.g., `PATH` in backticks).
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
✅ Deploy Preview for selenium-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Thanks for this contribution, @boris779! After syncing this branch with Since the documentation gap this PR set out to fix is already covered, I'm closing it. Thanks again for flagging the gap and for the PR! |
User description
added documentation for the argument skip-driver-in-path
Description
Motivation and Context
I missed the documentation for that, needed for Problems with snap geckodriver
Types of changes
Checklist
PR Type
Documentation
Description
Added documentation for the
--skip-driver-in-pathargument.Updated Selenium Manager documentation table with a new argument.
Changes walkthrough 📝
selenium_manager.en.md
Added `--skip-driver-in-path` argument documentationwebsite_and_docs/content/documentation/selenium_manager.en.md
--skip-driver-in-pathargument in the argumentstable.