-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Improve SSL Certificate Verification details #9477
Conversation
Hello! I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the |
This needs a news fragment. |
Any pointers on why CI is failing? |
I think you need to break the message into more lines, it's exceeding the length limit. |
I made it as short as other |
Co-authored-by: Tzu-ping Chung <[email protected]>
cert = partial(
PipOption,
- '--cert',
- dest='cert',
- type='path',
- metavar='path',
+ "--cert",
+ dest="cert",
+ type="path",
+ metavar="path", Not sure why the linter suddenly starts to care about quotes in this function 🤔 But let’s include the suggestion anyway, since we indeed prefer double quotes. |
Because this subpackage is finally enforced in CI to follow black -- see #9635. |
Pipeline fixed |
Based on details provided at #6720 (comment) by @chrahunt
--cert
help text that setting--cert
overrides default CA bundleFix #6720
Please use squash merge option when merging.