Skip to content

Certifi patch - #1566

Open
SohierDane wants to merge 2 commits into
mainfrom
certifi-patch
Open

SohierDane wants to merge 2 commits into
mainfrom
certifi-patch

Conversation

@SohierDane

Copy link
Copy Markdown
Contributor

For https://b.corp.google.com/issues/575927270

Test build is underway as of 13:49 PST.

@SohierDane SohierDane self-assigned this Oct 20, 2026
@SohierDane

Copy link
Copy Markdown
Contributor Author

CPU-only build finished and passed all tests. I'm not currently set up to run the GPU tests.

@SohierDane
SohierDane requested review from calderjo and rosbo October 20, 2026 20:57
@rosbo

rosbo commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Let's wait to merge until the CI build passes: https://ci.kaggle.net/job/kernels/job/docker-python/job/certifi-patch/

@SohierDane

Copy link
Copy Markdown
Contributor Author

Let's wait to merge until the CI build passes: https://ci.kaggle.net/job/kernels/job/docker-python/job/certifi-patch/

SGTM. I've also got a follow up PR in the works, might make sense to wait for it as well.

@SohierDane
SohierDane added this pull request to stack #1568 October 20, 2026 21:38
@rosbo rosbo closed this Oct 21, 2026
@rosbo rosbo reopened this Oct 21, 2026
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.

2 participants