-
Notifications
You must be signed in to change notification settings - Fork 120
package with pyproject.toml #253
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
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThis PR migrates packaging to pyproject/setuptools, removes setup.py/setup.cfg directives, updates Makefile and tox targets, adjusts bors and Nix flake config, adds Sphinx versioning from mpd.VERSION, tightens lxml import handling in a doc script, formats examples/tests, adds a logger in mpd.base, and introduces an example LockableMPDClient. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User
participant L as LockableMPDClient
participant T as Lock (threading.Lock)
participant C as MPDClient
User->>L: with LockableMPDClient() as client
activate L
L->>T: acquire()
note right of T: Serialize access
User->>L: client.command(args)
L->>C: delegate command(args)
C-->>L: result
L-->>User: result
User-->>L: exit context
L->>T: release()
deactivate L
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (25)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
8a64648 to
6a5b427
Compare
c85aea5 to
18eac05
Compare
2d5c412 to
2699197
Compare
2699197 to
6f6e543
Compare
Summary by CodeRabbit