add inline dependencies #269

Open
Alistair1231 wants to merge 2 commits from Alistair1231/inline-dependencies into main
Alistair1231 commented 2026-04-06 07:32:20 -04:00 (Migrated from github.com)

This commit adds inline dependencies which allows for tools like uv to create and manage the venv for you when run like this:

sudo uv run main.py

Docs of uv on the topic

This commit adds inline dependencies which allows for tools like uv to create and manage the venv for you when run like this: ```bash sudo uv run main.py ``` [Docs of uv on the topic](https://docs.astral.sh/uv/guides/scripts/#declaring-script-dependencies)
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2026-04-06 07:34:05 -04:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Pull request overview

This PR adds PEP 723-style inline script metadata to main.py so tools like uv can resolve and install the script’s Python dependencies automatically when running the script.

Changes:

  • Added a # /// script inline metadata block declaring tqdm, requests, and InquirerPy as dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Pull request overview This PR adds PEP 723-style inline script metadata to `main.py` so tools like `uv` can resolve and install the script’s Python dependencies automatically when running the script. **Changes:** - Added a `# /// script` inline metadata block declaring `tqdm`, `requests`, and `InquirerPy` as dependencies. --- 💡 <a href="/casualsnek/waydroid_script/new/main?filename=.github/instructions/*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.
@@ -2,0 +6,4 @@
# "dbus-python",
# "gbinder",
# "PyGObject",
# "InquirerPy"
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2026-04-06 07:34:04 -04:00

The inline # /// script metadata is parsed as TOML (PEP 723 style). In a TOML array, items must be comma-separated; as written, the dependencies = [ block is invalid and tools like uv run --script will fail to parse it. Add commas between the dependency strings (a trailing comma is also fine).

#   "InquirerPy",
The inline `# /// script` metadata is parsed as TOML (PEP 723 style). In a TOML array, items must be comma-separated; as written, the `dependencies = [` block is invalid and tools like `uv run --script` will fail to parse it. Add commas between the dependency strings (a trailing comma is also fine). ```suggestion # "InquirerPy", ```
@@ -2,0 +8,4 @@
# "PyGObject",
# "InquirerPy"
# ]
# ///
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2026-04-06 07:34:05 -04:00

This introduces a second source of truth for runtime dependencies (inline here and also in requirements.txt). That duplication can drift over time and cause confusing install/runtime mismatches. Consider either generating/deriving one from the other, or documenting which one is authoritative (and when each should be updated).


This introduces a second source of truth for runtime dependencies (inline here and also in `requirements.txt`). That duplication can drift over time and cause confusing install/runtime mismatches. Consider either generating/deriving one from the other, or documenting which one is authoritative (and when each should be updated). ```suggestion ```
Checking for merge conflicts…
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin Alistair1231/inline-dependencies:Alistair1231/inline-dependencies
git checkout Alistair1231/inline-dependencies
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wbruno/waydroid_script#269