Skip to content
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

Pin our importlib_metadata version (and some cosmetic fixes) #319

Merged
merged 3 commits into from
May 9, 2023

Conversation

jherland
Copy link
Member

(Depends on #313)

Commits:

  • tests.utils.assert_unordered_equivalence(): More details when it fails
  • test_invocation: Fix pytest.skip() message
  • pyproject.toml: Pin importlib_metadata to latest release ==v6.6.0

@jherland jherland requested review from mknorps and Nour-Mws April 27, 2023 01:04
@jherland jherland self-assigned this Apr 27, 2023
@jherland jherland force-pushed the jherland/multiple-pyenvs branch from 5e5904a to d03225a Compare May 5, 2023 23:44
@jherland jherland force-pushed the jherland/minor-fixes branch from a7f4c88 to 0b7d7ec Compare May 6, 2023 00:10
@jherland jherland force-pushed the jherland/multiple-pyenvs branch from d03225a to 1ade7d9 Compare May 6, 2023 01:02
@jherland jherland force-pushed the jherland/minor-fixes branch from 0b7d7ec to acb5882 Compare May 6, 2023 01:07
@jherland jherland force-pushed the jherland/multiple-pyenvs branch from 517775d to a10e43a Compare May 8, 2023 13:25
Base automatically changed from jherland/multiple-pyenvs to main May 8, 2023 13:50
@dpulls
Copy link

dpulls bot commented May 8, 2023

🎉 All dependencies have been resolved !

jherland added 3 commits May 8, 2023 17:16
v6.3.0 is the first version to include our fix for inferring top-level
import names in packages using the old-style .egg installation format.

However, there are more changes/fixes happening in importlib_metadata
these days, and given that we depend on it quite intimately (including
calling some non-publicized interfaces), we're probably better off
_pinning_ it in our pyproject.toml rather than allowing our development
environment to automatically encounter bugs/incompatibilities in the
latest version.
@jherland jherland force-pushed the jherland/minor-fixes branch from acb5882 to 1e3a1bd Compare May 8, 2023 15:19
@jherland jherland merged commit d815c0c into main May 9, 2023
@jherland jherland deleted the jherland/minor-fixes branch May 9, 2023 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants