Skip to content

Conversation

@oaimtiaz
Copy link
Contributor

@oaimtiaz oaimtiaz commented Jun 3, 2025

Changes

This PR corrects the package name used when retrieving the version number for sqlalchemy-citext. Previously, the code called:

version('citext')

which raised a PackageNotFoundError:

importlib.metadata.PackageNotFoundError: citext

Although sqlalchemy-citext can be imported as citext, the correct package identifier for importlib.metadata.version() is sqlalchemy-citext. This PR updates the call accordingly:

version('sqlalchemy-citext')

Checklist

If this is a user-facing code change, like a bugfix or a new feature, please ensure that
you've fulfilled the following conditions (where applicable):

  • You've added tests (in tests/) which would fail without your patch
  • You've added a new changelog entry (in CHANGES.rst).

If this is a trivial change, like a typo fix or a code reformatting, then you can ignore
these instructions.

@sheinbergon sheinbergon self-requested a review June 3, 2025 06:13
@sheinbergon sheinbergon added this to the 3.1 milestone Jun 3, 2025
@sheinbergon sheinbergon added bug dependencies Pull requests that update a dependency file labels Jun 3, 2025
@sheinbergon sheinbergon merged commit 1626080 into agronholm:master Jun 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants