Repository navigation
Package for PyPI: pip install minidsp-dash - #3
Merged
Merged
Conversation
The repo had an audience but nothing installable — no releases, no package, so the only way in was cloning and reading. This makes it `pip install minidsp-dash` with a `minidsp-dash` command on PATH. server.py is renamed to minidsp_dash.py so the import name matches the project name. That rename is the point rather than cosmetic: `py-modules = ["server"]` would drop a module called `server` into site-packages, where it could shadow or be shadowed by anything. Running from a checkout still works exactly as before, just under the new filename, and the README documents both paths. Dependencies stay empty on purpose — the whole pitch is a bare-stdlib single file, and the one real requirement, the minidsp CLI, is not a Python package so it cannot be expressed here anyway. Publishing uses Trusted Publishing (OIDC) rather than an API token, so no secret is stored in the repo or its settings; the workflow builds, runs twine check --strict, and publishes on a GitHub Release. Actions pinned to current majors (checkout v7, setup-python v7, upload-artifact v7, download-artifact v8). Verified locally: both artifacts pass twine check --strict, and the wheel installs into a clean venv where the console script lands on PATH, the module imports, and parse_status runs from the installed copy. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The repo had an audience but nothing installable — no releases, no package — so the only way in was cloning and reading. This makes it
pip install minidsp-dash, with aminidsp-dashcommand on PATH.The rename is the point, not cosmetic
server.pybecomesminidsp_dash.pyso the import name matches the project name.py-modules = ["server"]would drop a module calledserverinto site-packages, where it could shadow or be shadowed by anything else. Running from a checkout still works exactly as before, just under the new filename; the README documents both paths.No dependencies, deliberately
The pitch is a bare-stdlib single file. The one real requirement — the
minidspCLI — is not a Python package, so it cannot be expressed as one and has to be installed separately either way.Publishing
Trusted Publishing (OIDC), not an API token, so no secret lives in this repo or its settings. The workflow builds, runs
twine check --strict, and publishes on a GitHub Release. Actions pinned to current majors: checkout v7, setup-python v7, upload-artifact v7, download-artifact v8.One setup step is needed on PyPI before the first release — configure the publisher at the project's publishing settings with owner
av-dsp-tools, repositoryminidsp-dash, workflowpublish.yml, environmentpypi.Verified locally
twine check --strictparse_statusruns from the installed copyunittest discoverpasses against the renamed module🤖 Generated with Claude Code