Skip to content

Use WebDriver::managed in README and lib.rs example#306

Merged
stevepryde merged 1 commit intomainfrom
claude/funny-franklin-89511a
Apr 30, 2026
Merged

Use WebDriver::managed in README and lib.rs example#306
stevepryde merged 1 commit intomainfrom
claude/funny-franklin-89511a

Conversation

@stevepryde
Copy link
Copy Markdown
Owner

Summary

  • Switch the example in thirtyfour/README.md (and the top-level README.md symlink) from WebDriver::new("http://localhost:9515", caps) to WebDriver::managed(caps).
  • Update the crate-root rustdoc example in thirtyfour/src/lib.rs the same way, and rewrite the surrounding preamble: with WebDriver::managed the reader no longer has to start chromedriver on a port — it auto-downloads, runs, and tears down for them.

Why

The book, the in-tree examples, and the WebDriver::managed rustdoc all already lead with the managed flow. The README and crate-root docs were the last surface still pointing newcomers at the manual chromedriver-on-a-port path, which made the recommended flow look optional. Now first impressions match the rest of the docs.

Test plan

  • cargo doc --no-deps --all-features (intra-doc link to WebDriver::managed resolves cleanly)
  • Render the README on GitHub to confirm the snippet still looks right

🤖 Generated with Claude Code

The crate's docs in the book and examples already lead with
WebDriver::managed; the top-level README and crate-root rustdoc were
still showing the chromedriver-on-a-port flow. Switch them over so the
first thing a new reader sees matches the recommended path.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@stevepryde stevepryde merged commit e0f3905 into main Apr 30, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant