Skip to content
This repository has been archived by the owner on Feb 26, 2023. It is now read-only.

Take any version of cerifi #73

Merged
merged 1 commit into from
Jun 22, 2022
Merged

Take any version of cerifi #73

merged 1 commit into from
Jun 22, 2022

Conversation

jonringer
Copy link
Contributor

Certifi has a stable api, only the certificate contents get changed.

So it's safe to assume the the latest is always the version you want.

Certifi has a stable api, only the certificate contents get changed.

So it's safe to assume the the latest is always the version you want.
@cgarciae
Copy link
Owner

Hey @jonringer, thanks for the PR! Seems like a good idea, the dependency on certifi was added just to please the environment on CI, so we can be al lax about the version as needed.

pre-commit is failing but this is not due to your change, I'll merge it and fix it later.

@cgarciae cgarciae merged commit 2b6ba0a into cgarciae:master Jun 22, 2022
@jonringer jonringer deleted the patch-1 branch June 22, 2022 23:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants