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

Remove urllib3 constraint #337

Closed
wants to merge 1 commit into from
Closed

Conversation

CasperWA
Copy link
Contributor

Description:

Closes #328

In order to test this, install latest 'urllib3' after installing everything else, as this will otherwise clash with the requirements of oteapi-core.

Type of change:

  • Bug fix.
  • New feature.
  • Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

  • Is the change limited to one issue?
  • Does this PR close the issue?
  • Is the code easy to read and understand?
  • Do all new feature have an accompanying new test?
  • Has the documentation been updated as necessary?

In order to test this, install latest 'urllib3' after installing
everything else, as this will otherwise clash with the requirements of
oteapi-core.
@codecov-commenter
Copy link

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (6635192) 69.15% compared to head (ed14791) 69.15%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #337   +/-   ##
=======================================
  Coverage   69.15%   69.15%           
=======================================
  Files          18       18           
  Lines         522      522           
=======================================
  Hits          361      361           
  Misses        161      161           
Flag Coverage Δ
pytest 69.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@CasperWA CasperWA closed this Oct 10, 2023
@CasperWA CasperWA deleted the cwa/remove-urllib3-constraint branch October 10, 2023 12:50
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.

Remove upper bound on urllib3
2 participants