Skip to content

PEP-621 compliant project metadata & build configuration #302

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 23, 2025
Merged

PEP-621 compliant project metadata & build configuration #302

merged 5 commits into from
Sep 23, 2025

Conversation

@alisaifee
Copy link
Owner

@alisaifee alisaifee commented Sep 17, 2025 •
edited
Loading

Description

  • Migrate most/all project metadata to pyproject.toml
  • Migrate all extras and dependency groups away from requirements/*.txt to pyproject.toml
  • Use uv for dependency management and as a build frontend

@codecov
Copy link

codecov bot commented Sep 17, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.51%. Comparing base (96320f5) to head (226cf1a).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #302      +/-   ##
==========================================
- Coverage   95.65%   95.51%   -0.15%     
==========================================
  Files          81       81              
  Lines       10597    10597              
==========================================
- Hits        10137    10122      -15     
- Misses        460      475      +15

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alisaifee alisaifee requested a review from Graeme22 September 17, 2025 23:42
@alisaifee alisaifee marked this pull request as ready for review September 17, 2025 23:43
@alisaifee alisaifee force-pushed the pep-621 branch 5 times, most recently from d6610d0 to c2a50eb Compare September 18, 2025 01:40
@hyperlint-ai
Copy link

hyperlint-ai bot commented Sep 18, 2025

PR Change Summary

Migrated project metadata and dependency management to PEP-621 compliant configuration.

  • Migrated project metadata to pyproject.toml
  • Moved dependency groups from requirements/*.txt to pyproject.toml
  • Implemented uv for dependency management and build frontend
  • Updated test command to use uv instead of pip

Modified Files

  • docs/source/handbook/development.rst
How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@alisaifee alisaifee force-pushed the pep-621 branch 2 times, most recently from ef24b0f to 196dc70 Compare September 18, 2025 02:48
@alisaifee
Migrate project metadata to pyproject
aae59cb 
- Move all project metadata to pyproject.toml
- Use uv build system
Graeme22
Graeme22 approved these changes Sep 19, 2025
View reviewed changes
Copy link
Collaborator

@Graeme22 Graeme22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, especially since everything is building correctly!

@alisaifee alisaifee merged commit b169b67 into master Sep 23, 2025
108 of 109 checks passed
@alisaifee alisaifee deleted the pep-621 branch September 23, 2025 14:48
Graeme22 added a commit to Graeme22/coredis that referenced this pull request Sep 26, 2025
@Graeme22
Combine commits for anyio upgrade
8c6dfa1 
replace sleeps

clean up error handling

remove futures from basic client

update connections to use anyio

lazy processing of responses

fix edge cases

pubsub now working

add max idle time

small tweaks

revert lazy processing, use context managers everywhere

pubsub uses strict async context manager

update pubsub tests

blocking pool working

add pipelining and scripting

clean up pubsub a bit

handle blocking connections for pubsub/pipelines/blocking commands

restructure notifications for blocking pool

more reliable transactions (from redis-py)

tweak connection allocation logic

fix race condition

remove monitor, small fixes

guard connection after close

fix on_connect

log connection bug

add diagnostics for git

fix bug

catch error

add logger

idle connections cleanup gracefully, update more tests

update more tests, work on sentinel

fix sentinel bugs

small optimizations

Bump sphinxext-opengraph from 0.10.0 to 0.12.0 (alisaifee#293)

Bumps [sphinxext-opengraph](https://github.com/sphinx-doc/sphinxext-opengraph) from 0.10.0 to 0.12.0.
- [Release notes](https://github.com/sphinx-doc/sphinxext-opengraph/releases)
- [Commits](sphinx-doc/sphinxext-opengraph@v0.10.0...v0.12.0)

---
updated-dependencies:
- dependency-name: sphinxext-opengraph
  dependency-version: 0.12.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Ensure ssl_context from kwargs is respected when using from_url factory method

Bump sphinxext-opengraph from 0.12.0 to 0.13.0 (alisaifee#297)

Bumps [sphinxext-opengraph](https://github.com/sphinx-doc/sphinxext-opengraph) from 0.12.0 to 0.13.0.
- [Release notes](https://github.com/sphinx-doc/sphinxext-opengraph/releases)
- [Commits](sphinx-doc/sphinxext-opengraph@v0.12.0...v0.13.0)

---
updated-dependencies:
- dependency-name: sphinxext-opengraph
  dependency-version: 0.13.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump sphinx-sitemap from 2.7.2 to 2.8.0 (alisaifee#296)

Bumps [sphinx-sitemap](https://github.com/jdillard/sphinx-sitemap) from 2.7.2 to 2.8.0.
- [Release notes](https://github.com/jdillard/sphinx-sitemap/releases)
- [Changelog](https://github.com/jdillard/sphinx-sitemap/blob/master/CHANGELOG.rst)
- [Commits](jdillard/sphinx-sitemap@v2.7.2...v2.8.0)

---
updated-dependencies:
- dependency-name: sphinx-sitemap
  dependency-version: 2.8.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ali-Akber Saifee <[email protected]>

Update changelog for  5.1.0

Bump mypy from 1.17.1 to 1.18.1 (alisaifee#299)

Bumps [mypy](https://github.com/python/mypy) from 1.17.1 to 1.18.1.
- [Changelog](https://github.com/python/mypy/blob/master/CHANGELOG.md)
- [Commits](python/mypy@v1.17.1...v1.18.1)

---
updated-dependencies:
- dependency-name: mypy
  dependency-version: 1.18.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Switch to bitnamilegacy for redis-sentinel

Gracefully handle MODULE LIST error (alisaifee#301)

PEP-621 compliant project metadata & build configuration (alisaifee#302)

- Move all project metadata to pyproject.toml
- Use uv build system

Fix error in linting step in compatibility workflow

Add verbose to pypi upload step

Fix pure python build step

fix pyproject

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

Reviewers

@Graeme22 Graeme22 Graeme22 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alisaifee @Graeme22