-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix pkgresources import #7866
Fix pkgresources import #7866
Conversation
upgrade-note: "If you had previously constrained `setuptools<71` in your `pyproject.yml`, you may now drop the constraint" | ||
destination-branches: | ||
- master | ||
- iso7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iso6 will have its own PR because it has a second occurrence of this import.
@edvgui requesting your review for the changelog entries (what's in "sections" will be visible to the customers in the generated changelog for the patch release in the documentation). |
@arnaudsjs I had to apply one more fix due to pypa/setuptools#4482. It's not pretty but it's the best I can think of on short notice, and we can hopefully get rid of the hack when we migrate away from |
Processing this pull request |
# Description Fixed import of `pkg_resources.extern`, which was dropped in the latest `setuptools` release (v71). # Self Check: Strike through any lines that are not applicable (`~~line~~`) then check the box - [x] ~~Attached issue to pull request~~ - [x] Changelog entry - [x] ~~Type annotations are present~~ - [x] ~~Code is clear and sufficiently documented~~ - [x] No (preventable) type errors (check using make mypy or make mypy-diff) - [x] ~~Sufficient test cases (reproduces the bug/tests the requested feature)~~ - [x] Correct, in line with design - [x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: ) - [x] ~~If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)~~
Merged into branches master in 2ce0148 |
# Description Fixed import of `pkg_resources.extern`, which was dropped in the latest `setuptools` release (v71). # Self Check: Strike through any lines that are not applicable (`~~line~~`) then check the box - [x] ~~Attached issue to pull request~~ - [x] Changelog entry - [x] ~~Type annotations are present~~ - [x] ~~Code is clear and sufficiently documented~~ - [x] No (preventable) type errors (check using make mypy or make mypy-diff) - [x] ~~Sufficient test cases (reproduces the bug/tests the requested feature)~~ - [x] Correct, in line with design - [x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: ) - [x] ~~If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)~~
Processing #7868. |
Pull request opened by the merge tool on behalf of #7866
# Description Fixed import of `pkg_resources.extern`, which was dropped in the latest `setuptools` release (v71). # Self Check: Strike through any lines that are not applicable (`~~line~~`) then check the box - [x] ~~Attached issue to pull request~~ - [x] Changelog entry - [x] ~~Type annotations are present~~ - [x] ~~Code is clear and sufficiently documented~~ - [x] No (preventable) type errors (check using make mypy or make mypy-diff) - [x] ~~Sufficient test cases (reproduces the bug/tests the requested feature)~~ - [x] Correct, in line with design - [x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: ) - [x] ~~If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)~~
# Description Fixed import of `pkg_resources.extern`, which was dropped in the latest `setuptools` release (v71). # Self Check: Strike through any lines that are not applicable (`~~line~~`) then check the box - [x] ~~Attached issue to pull request~~ - [x] Changelog entry - [x] ~~Type annotations are present~~ - [x] ~~Code is clear and sufficiently documented~~ - [x] No (preventable) type errors (check using make mypy or make mypy-diff) - [x] ~~Sufficient test cases (reproduces the bug/tests the requested feature)~~ - [x] Correct, in line with design - [x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: ) - [x] ~~If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)~~
Description
Fixed import of
pkg_resources.extern
, which was dropped in the latestsetuptools
release (v71).Self Check:
Strike through any lines that are not applicable (
~~line~~
) then check the boxAttached issue to pull requestType annotations are presentCode is clear and sufficiently documentedSufficient test cases (reproduces the bug/tests the requested feature)If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)