Upgrade of setuptools with fixes to dependencies#14330
Conversation
giohappy
changed the title
giohappy
changed the title
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates several dependency versions in pyproject.toml, including django-downloadview, django-polymorphic, geonode-pinax-notifications, geonode-oauth-toolkit, and setuptools. Feedback is provided regarding the setuptools dependency constraint, pointing out that removing the lower bound could allow incompatible older versions to be installed, and noting that the adjacent comment is now outdated and should be updated.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| # importer dependencies | ||
| # blocked to <80 to avoid warning during the runtime for with py312 related to the polymorphic model | ||
| "setuptools>=59,<80", | ||
| "setuptools<83", |
There was a problem hiding this comment.
The comment on line 111 is now outdated and contradicts the new constraint of <83. Please update or remove the comment to reflect this change.
Additionally, removing the lower bound (>=59) might allow older, incompatible versions of setuptools to be installed at runtime. It is recommended to retain a lower bound of >=61.0 to align with the build-system requirement.
| "setuptools<83", | |
| "setuptools>=61.0,<83", |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #14330 +/- ##
===========================================
+ Coverage 34.71% 74.92% +40.21%
===========================================
Files 982 982
Lines 60496 60538 +42
Branches 8247 8258 +11
===========================================
+ Hits 21002 45360 +24358
+ Misses 38332 13348 -24984
- Partials 1162 1830 +668 🚀 New features to boost your workflow:
|
|
Let's wait for #14339 then we will run the tests again |
Bumps [cryptography](https://github.com/pyca/cryptography) from 48.0.0 to 49.0.0. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@48.0.0...49.0.0) --- updated-dependencies: - dependency-name: cryptography dependency-version: 49.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* Upgrade of setupttols to 82 with fixes to dependencies * Sanitize metadata input * build(deps): bump cryptography from 48.0.0 to 49.0.0 Bumps [cryptography](https://github.com/pyca/cryptography) from 48.0.0 to 49.0.0. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@48.0.0...49.0.0) --- updated-dependencies: - dependency-name: cryptography dependency-version: 49.0.0 dependency-type: direct:production update-type: version-update:semver-major ... * [Fixes #14339] Drop support Dropbox storage manager --------- (cherry picked from commit 3d929fc) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Giovanni Allegri <giohappy@gmail.com> Co-authored-by: Emanuele Tajariol <etj@geo-solutions.it> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Mattia Giupponi <mattia.giupponi@gmail.com>
3 participants
No description provided.