Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32.4k
gh-110964: clinic: pass clinic argument to bad_argument()#110984
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The only purpose of this change is to avoid usingclinic global variable in |
AlexWaygood commentedOct 17, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Please give me some time to look at this! :-) |
"Check if generated files are up to date (pull_request)" failure looks unrelated:
|
Don't rely on the global 'clinic' argument: pass explicitly a'clinic' argument.
I rebased my PR on the main branch to try to fix the "Check if generated files are up to date" job failure. |
Mmmmh, so do you need more time to review? |
Well, since@AlexWaygood is not available for review, i think that i will just merge my change next days. |
AlexWaygood commentedNov 3, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Thanks for waiting, sorry for the slow response from me! I wanted to take the time to think about this properly, but haven't had a chance recently due to being busy at work, and it's unlikely that I'll have the time in the next week either. My instinct is that there should be a simpler solution here that involves fewer changes, but if you feel like there's a rush to get this merged for whatever reason, then please go ahead. I agree I've kept you waiting for a while :-) |
me:
Yeah, there should be a way to redesign the code to have to pass less arguments. I dislike the current API :-( Moreover, passing limited_capi and clinic is kind of redundant, since using I close my PR for now. We can revisit this code later when we will try to give rid of the global |
Uh oh!
There was an error while loading.Please reload this page.
Don't rely on the global 'clinic' argument: pass explicitly a 'clinic' argument.