Skip to content

Conversation

@bidoubiwa
Copy link
Contributor

@bidoubiwa bidoubiwa commented Nov 30, 2022

The chromedriver version required by docs-scraper was not compatible with the chromedriver version in the previous python image 3.8.4 in the DockerFile.

Updating to latest python:3.10.8-slim-bullseye fixes this issue.

Additionally, the requests package is not a standard library so it should be explicitly added to the Pipfile.

We did not update to python:3.11.0-slim-bullseye because selenium is using a package that is not compatible with 3.11. This will be fixed in another issue. See #285

@bidoubiwa bidoubiwa added the bug Something isn't working label Nov 30, 2022
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much @bidoubiwa for this PR ❤️
LGTM!

@alallema alallema merged commit 55ed18c into main Nov 30, 2022
@alallema alallema deleted the update_docker_file_to_new_python_version branch November 30, 2022 16:20
Copy link
Member

@brunoocasali brunoocasali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants