Skip to content

Conversation

@alexlyzhov
Copy link
Contributor

@alexlyzhov alexlyzhov commented Oct 20, 2025

User description

Updated certifi dependency to the latest version.


PR Type

Enhancement


Description

  • Removed version pinning from certifi dependency

  • Allows certifi to use any compatible version

  • Simplifies dependency management and updates


Diagram Walkthrough

flowchart LR
  A["setup.py"] -- "certifi~=2025.4.26" --> B["Pinned Version"]
  A -- "certifi" --> C["Any Compatible Version"]
Loading

File Walkthrough

Relevant files
Dependencies
setup.py
Remove certifi version pinning constraint                               

setup.py

  • Changed certifi dependency from pinned version certifi~=2025.4.26 to
    unpinned certifi
  • Removes version constraint allowing any compatible certifi version to
    be installed
  • Simplifies dependency management by delegating version selection to
    pip resolver
+1/-1     

Updated certifi dependency to the latest version.
@qodo-merge-pro
Copy link

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🎫 No ticket provided
- [ ] Create ticket/issue <!-- /create_ticket --create_ticket=true -->

</details></td></tr>
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
No custom compliance provided

Follow the guide to enable custom compliance check.

Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-merge-pro
Copy link

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
Pin dependency to a compatible range

Re-introduce a version constraint for the certifi dependency by setting a lower
bound, such as certifi>=2025.4.26. This prevents potential issues from future
breaking changes in the dependency.

setup.py [15]

-"certifi",
+"certifi>=2025.4.26",
  • Apply / Chat
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly identifies the risk of unpinning the certifi dependency and proposes a sensible lower-bound pin to ensure future compatibility and stability.

Medium
  • More

@pjoshi30
Copy link
Contributor

@alexlyzhov can you test this by installing a local package? Last time an attempt to upgrade one of these dependencies broke certain things.

@alexlyzhov
Copy link
Contributor Author

@pjoshi30 I tested it thoroughly, works well, we can merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants