-
Notifications
You must be signed in to change notification settings - Fork 305
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
Fix failing python tests #778
Conversation
@Daverlo could you review this. I believe you originally wrote these python tests. With these changes, I am not sure if the workflows are still testing the things you are intending them to test. |
Was the |
Yes...let me remove that. |
The requests module 1.2.3 is no longer compatible with python >=3.10.0. So, instead use 2.26.0, the most recent version that is compatible with our supported python versions at the time of this commit.
1b16d54
to
3df2172
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine to me. I wonder if we could leave the "*"
version specifiers unchanged.
@@ -6,7 +6,7 @@ verify_ssl = true | |||
[dev-packages] | |||
|
|||
[packages] | |||
requests = "*" | |||
requests = "2.26.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this change could be reverted, hopefully the version from the lock file will be taken.
@@ -6,7 +6,7 @@ verify_ssl = true | |||
[dev-packages] | |||
|
|||
[packages] | |||
requests = "*" | |||
requests = "2.26.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this change could be reverted, hopefully the version from the lock file will be taken.
@@ -6,6 +6,6 @@ verify_ssl = true | |||
[dev-packages] | |||
|
|||
[packages] | |||
requests = "*" | |||
requests = "2.26.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this change could be reverted, hopefully the version from the lock file will be taken.
@@ -6,7 +6,7 @@ authors = ["Your Name <you@example.com>"] | |||
|
|||
[tool.poetry.dependencies] | |||
python = "^3.8" | |||
requests = "*" | |||
requests = "2.26.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this change could be reverted, hopefully the version from the lock file will be taken.
python = "^3.5" | ||
requests = "*" | ||
python = "^3.6" | ||
requests = "2.26.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this change could be reverted, hopefully the version from the lock file will be taken.
requests = "2.26.0" | |
requests = "*" |
Also, rename test scripts to indicate the version they are using.
67eb85e
to
43ea2ce
Compare
The requests module 1.2.3 is no longer compatible with python >=3.10.0.
So, instead use 2.26.0, the most recent version that is compatible with
our supported python versions at the time of this commit.
Merge / deployment checklist