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 note about new tabs/windows to Troubleshooting #452

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

jkrasnay
Copy link
Contributor

@jkrasnay jkrasnay commented Jun 2, 2022

No description provided.

@lread lread merged commit f3f0370 into clj-commons:master Jun 2, 2022
lread added a commit to kidd/etaoin that referenced this pull request Jun 15, 2022
* master: (94 commits)
  Switch with-exception from private to no-doc (clj-commons#457)
  API review: docstrings, arglists, publics (clj-commons#456)
  User guide and usage related updates (clj-commons#454)
  docs: changelog [skip ci]
  docs: README: update contributors [skip ci]
  Add note about new tabs/windows to Troubleshooting (clj-commons#452) [skip ci]
  Review and update Dockerfile (clj-commons#451)
  lint: tweak .gitignore [skip ci]
  lint: abandon idea of linting clj-kondo hooks
  lint: lint clj-kondo hooks code too
  lint: update dev guide
  lint: distinguish internal hooks from imports
  lint: bump clj-kondo
  lint: Use bb's (clojure) instead of shelling out
  Add clj-kondo export config
  Update: cheshire 5.11.0 was just released
  Bump dependencies to current releases
  docs: update changelog [skip ci]
  docs: add Dave Yarwood to contributors [skip ci]
  Move `effectively-displayed?` docstring to the public `displayed-el?` function
  ...
lread added a commit to verma/etaoin that referenced this pull request Jun 16, 2022
Ya I see it:
- running? and wait-running very likely don't mean much for for a remote hosted
  WebDriver service as they only check the host and port for reachability.
  We could extract host and port from a the webdriver-url but checking
  that we can open a socket on port 443 for chrome.browserless.io
  doesn't say much about WebDriver service running at
  https://chrome.browserless.io/webdriver.

Minor changes:
- documented in user guide
- added a test
- minor strategy change around wait-running
- check :webdriver-url for nil rather than via str/blank? (for
  consistency with other options)

* master: (97 commits)
  docs: changelog [skip ci]
  Add new examples in getting started (clj-commons#364)
  Switch with-exception from private to no-doc (clj-commons#457)
  API review: docstrings, arglists, publics (clj-commons#456)
  User guide and usage related updates (clj-commons#454)
  docs: changelog [skip ci]
  docs: README: update contributors [skip ci]
  Add note about new tabs/windows to Troubleshooting (clj-commons#452) [skip ci]
  Review and update Dockerfile (clj-commons#451)
  lint: tweak .gitignore [skip ci]
  lint: abandon idea of linting clj-kondo hooks
  lint: lint clj-kondo hooks code too
  lint: update dev guide
  lint: distinguish internal hooks from imports
  lint: bump clj-kondo
  lint: Use bb's (clojure) instead of shelling out
  Add clj-kondo export config
  Update: cheshire 5.11.0 was just released
  Bump dependencies to current releases
  docs: update changelog [skip ci]
  ...
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.

2 participants