-
Notifications
You must be signed in to change notification settings - Fork 216
fix(jira): change search_issues() to extended_search_issues() #1125
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
Conversation
|
Looks like the CI failure is simply due to jira-3.10.x not supporting python-3.9. There are two possible solutions:
However, I wonder if upgrading the jira version was actually part of the solution for you? Generally the latest version gets installed regardless of the minimum requirement. Edit: I'd overlooked your comment about checking 0.22 before. I think changing to 3.9.4 and seeing if CI passes makes the most sense. |
It's possible that a later library version has simply fixed this underneath (although that wasn't my reading of the issue). I'll try 3.9.4. Again though, i'm leaving this draft at the moment because I haven't had time to test it out end to end in production. It worked for me as a hot fix to jira.py as i mentioned. |
|
Oh and "I don't know who needs to hear this and do the same" but I’m bumping our (modest) sponsorship of @gothenburgbitfactory now that bugwarrior is here too! |
|
3.10.0 is required to get the enhanced api https://github.com/pycontribs/jira/releases/tag/3.10.0
|
|
Ah, I see now that that's when jql support was added. So I'd say:
|
|
okay. I still haven't been able to package this and test locally. but i've tested this same 1-line change locally as mentioned. |
|
Could use a rebase since #1123 was merged. |
Think I merged but will rebase |
…y3.9 - also change dep for jira>=3.10.5 - drop python 3.9, bump jira>=3.10.0 Fixes GothenburgBitFactory#1124 Fixes: GothenburgBitFactory#1124
8c00c92 to
89a4954
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.
I hadn't noticed that this was undrafted. Ready to merge?
i think so… ! I didn't do the full local testing of this commit but i have tested the patch thoroughly as mentioned. maybe a follow up to do loop-with-token in case there are more than 5k issues… |
|
post script: i finally rebuilt from this repo, and |
|
I got the same deprecation warning from Jira; this change leads to a different error for me: Looking at https://github.com/pycontribs/jira/releases/tag/3.10.0 / pycontribs/jira#2326, I think the new function is simply called differently: |
You are right! See #1131 - I am not sure how this happened, but I was not testing what I thought I was testing. |
also change dep for jira>=3.10.5 (may be able to be earlier, did not check - 0.22 did not have the 'new' api)
tested this locally as a hotpatch
need to test this from packaged build
Fixes: Jira failing with deprecation error
Fixes #1124