Skip to content

fix: Temporarily add certifi<=2025.1.31 dependency#1183

Merged
Pijukatel merged 1 commit into
masterfrom
pin-certifi
May 2, 2025
Merged

fix: Temporarily add certifi<=2025.1.31 dependency#1183
Pijukatel merged 1 commit into
masterfrom
pin-certifi

Conversation

@Pijukatel

Copy link
Copy Markdown
Collaborator

Description

Temporarily add bounded certifi dependency until certification issues on latest release of certifi are fully resolved.
(certifi was indirect dependency of crawlee that was not version bounded by any of the direct dependencies).

Issues

@github-actions github-actions Bot added this to the 113rd sprint - Tooling team milestone May 1, 2025
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label May 1, 2025
@Pijukatel Pijukatel marked this pull request as ready for review May 1, 2025 08:15
@Pijukatel Pijukatel requested review from janbuchar and vdusek May 1, 2025 08:15

@vdusek vdusek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Pijukatel Pijukatel merged commit 25ff961 into master May 2, 2025
@Pijukatel Pijukatel deleted the pin-certifi branch May 2, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants