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

Pin urllib3 and requests. #121

Merged
merged 1 commit into from May 11, 2023
Merged

Pin urllib3 and requests. #121

merged 1 commit into from May 11, 2023

Conversation

pszulczewski
Copy link
Contributor

@pszulczewski pszulczewski commented May 11, 2023

requests doesn't support urllib3 2.0 yet see: psf/requests#6432
They don't use poetry for version constraints in pyproject.toml, so when our poetry.lock is updated it bumps up urllib3 to 2.0.2.

Pinning urllib3 in our pyproject.toml like in their setup.cfg.

@@ -89,4 +91,3 @@ build-backend = "poetry.core.masonry.api"
[tool.pytest.ini_options]
testpaths = "tests/"
addopts = "-vv"
python_paths = "./"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is used for django-pytest plugin and is causing warning in pytest. We don't use that plugin here.

Copy link
Contributor

@joewesch joewesch left a comment

Choose a reason for hiding this comment

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

Let's make sure to create an issue to track the upstream library and revert this when it is no longer needed.

@pszulczewski
Copy link
Contributor Author

Let's make sure to create an issue to track the upstream library and revert this when it is no longer needed.

#122

@pszulczewski pszulczewski merged commit 056b2a3 into develop May 11, 2023
24 checks passed
@pszulczewski pszulczewski deleted the pin-urllib3 branch May 11, 2023 14:01
jamesharr pushed a commit to jamesharr/pynautobot that referenced this pull request Jun 15, 2023
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.

None yet

2 participants