Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR migrates the project’s packaging and CI workflows to use uv, updates documentation to match the new install/run flow, and adjusts packaging metadata to support a parallax CLI entrypoint.
Changes:
- Switch build backend to Hatchling and add
uvsource configuration for a Git-sourced dependency. - Update GitHub Actions workflows (CI, pytest, publish) to use
uvfor environment setup and commands. - Update README install/run instructions, bump version, and add
.venv/to.gitignore.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
pyproject.toml |
Migrates build backend to Hatchling, adds uv sources, defines console script, and reorganizes extras/config. |
parallax/main_window.py |
Saves window size back to config on close. |
parallax/__main__.py |
Refactors into a main() function to support a console script entrypoint. |
parallax/__init__.py |
Bumps package version to 1.17.0. |
README.md |
Updates installation/running/developer instructions to use uv. |
.gitignore |
Switches ignored venv directory from venv/ to .venv/. |
.github/workflows/tag_and_publish.yml |
Uses uv for Python setup/version extraction/build and enables workflow_call. |
.github/workflows/pytest.yml |
Uses uv sync/uv run for installing deps and running tests with coverage gate. |
.github/workflows/ci.yml |
Uses uv for installing linters and running Ruff checks; updates docs build steps similarly. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pip install parallax-app will fails callling sfm which is from github, not from pypi
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jsiegle
approved these changes
May 1, 2026
Collaborator
jsiegle
left a comment
jsiegle
left a comment
There was a problem hiding this comment.
This looks good, glad you're switching to uv!
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
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.
Documentation: Updated the README.md with new instructions for installing and running the project using uv.
Configuration: Updated pyproject.toml to align with uv standards (including uv.sources for local/internal dependencies).
CI/CD: Modified the GitHub Actions workflow to utilize uv
Environment: Added .venv to .gitignore and bumped the project version.
TODO: I will add mypy.