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

Update readme for urllib3 import error #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ virtualenv venv
pip install -r requirements_dev.txt

# Create a local installation to include test dependencies
pip install -e .[test]
# Or if saw error messages like "zsh: no matches found: .[test]", try:
pip install -e '.[test]'
```

Expand All @@ -37,6 +35,9 @@ functionality exist in the file `ray_beam_runner/portability/ray_runner_test.py`
pytest ray_beam_runner/portability/ray_runner_test.py
```

If you got `ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'OpenSSL XXXXX'. See: https://github.com/urllib3/urllib3/issues/2168`, following instructions on the given URL might solve. You can either have `OpenSSL` updated or explicitly restrict version of `urllib3` library.


To run all local unit tests, you can simply run `pytest` from the root directory.

### Found issues? Want to help?
Expand Down