Skip to content

Commit c884332

Browse files
authored
fix(python): clean up setuptools usage and drop redundant/unnecessary lock files (github#449)
* Rely on `pyproject.toml` Use of `setup.py` is redundant. This leads to more accurate metadata: ```diff 17c17 < Requires-Python: >=3.8 --- > Requires-Python: >=3.9 29d28 < Dynamic: requires-python ``` * Drop unnecessary/redundant lock files * Don't require a lock file
1 parent 93a3f71 commit c884332

File tree

6 files changed

+7
-600
lines changed

6 files changed

+7
-600
lines changed

.github/workflows/python-sdk-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ jobs:
5959
enable-cache: true
6060

6161
- name: Install Python dev dependencies
62-
run: uv sync --locked --all-extras --dev
62+
run: uv sync --all-extras --dev
6363

6464
- name: Install Node.js dependencies (for CLI in tests)
6565
working-directory: ./nodejs

python/.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -163,6 +163,9 @@ cython_debug/
163163
.ruff_cache/
164164
.ty_cache/
165165

166+
# uv
167+
uv.lock
168+
166169
# Build script caches
167170
.cli-cache/
168171
.build-temp/

python/pyproject.toml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,6 @@ dependencies = [
3232
Homepage = "https://github.com/github/copilot-sdk"
3333
Repository = "https://github.com/github/copilot-sdk"
3434

35-
[tool.setuptools.packages.find]
36-
where = ["."]
37-
include = ["copilot*"]
38-
3935
[project.optional-dependencies]
4036
dev = [
4137
"ruff>=0.1.0",
@@ -46,6 +42,9 @@ dev = [
4642
"httpx>=0.24.0",
4743
]
4844

45+
[tool.setuptools]
46+
packages = ["copilot"]
47+
4948
[tool.ruff]
5049
line-length = 100
5150
target-version = "py39"

python/setup.py

Lines changed: 0 additions & 11 deletions
This file was deleted.

python/test-requirements.txt

Lines changed: 0 additions & 5 deletions
This file was deleted.

0 commit comments

Comments
 (0)