Skip to content

chore: remove requirements.txt in favour of pyproject.toml #247

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.

Sign up for GitHub

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

Merged
mwbrooks merged 2 commits into from
Oct 19, 2026
Merged

chore: remove requirements.txt in favour of pyproject.toml #247

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1b94939
chore: remove requirements.txt in favour of pyproject.toml
mwbrooks Oct 19, 2026
34d7f5e
Merge remote-tracking branch 'origin/main' into mwbrooks-pyproject-de…
mwbrooks Oct 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .claude/CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ openai-agents-sdk/ # OpenAI Agents SDK implementation
pydantic-ai/ # Pydantic AI implementation
```

CI runs ruff lint/format checks against all directories via a matrix strategy in `.github/workflows/ruff.yml`. Dependabot monitors `requirements.txt` in all directories independently.
CI runs ruff lint/format checks against all directories via a matrix strategy in `.github/workflows/ruff.yml`. Dependabot monitors `pyproject.toml` in all directories independently.

## Architecture (shared across all implementations)

Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
- name: Install dependencies
run: |
pip install -U pip
pip install -r requirements.txt
pip install -e ".[dev]"
- name: Lint and check formatting
run: |
ruff check .
Expand All @@ -54,7 +54,6 @@ jobs:
- name: Install dependencies
run: |
pip install -U pip
pip install -r requirements.txt
pip install ".[test]"
- name: Run all tests
run: |
Expand Down
2 changes: 1 addition & 1 deletion claude-agent-sdk/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ source .venv/bin/activate # for Windows OS, .\.venv\Scripts\Activate instead sh
#### Install dependencies

```sh
pip install -r requirements.txt
pip install -e ".[dev]"
```

## Providers
Expand Down
2 changes: 1 addition & 1 deletion claude-agent-sdk/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dependencies = [
"slack-bolt==1.30.0",
"slack-cli-hooks==0.3.0",
"claude-agent-sdk==0.2.165",
"aiohttp==3.14.3",
"aiohttp==3.14.5",

@mwbrooks mwbrooks Oct 19, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: This version was behind requirements.txt. Dependabot bumps each file in separate PRs and some of the pyproject.toml bumps never landed. Removing requirements.txt leaves one file to update, which stops the drift.

"python-dotenv==1.2.4",
]

Expand Down
10 changes: 0 additions & 10 deletions claude-agent-sdk/requirements.txt
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion openai-agents-sdk/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ source .venv/bin/activate # for Windows OS, .\.venv\Scripts\Activate instead sh
#### Install dependencies

```sh
pip install -r requirements.txt
pip install -e ".[dev]"
```

## Providers
Expand Down
8 changes: 0 additions & 8 deletions openai-agents-sdk/requirements.txt
View file
Open in desktop

This file was deleted.

2 changes: 1 addition & 1 deletion pydantic-ai/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ source .venv/bin/activate # for Windows OS, .\.venv\Scripts\Activate instead sh
#### Install dependencies

```sh
pip install -r requirements.txt
pip install -e ".[dev]"
```

## Providers
Expand Down
2 changes: 1 addition & 1 deletion pydantic-ai/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dependencies = [
"slack-sdk==3.45.0",
"slack-bolt==1.30.0",
"slack-cli-hooks==0.3.0",
"pydantic-ai[openai]==2.54.0",
"pydantic-ai[openai,anthropic]==2.55.0",

@mwbrooks mwbrooks Oct 19, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: This version was behind requirements.txt. Dependabot bumps each file in separate PRs and some of the pyproject.toml bumps never landed. Removing requirements.txt leaves one file to update, which stops the drift. The anthropic extra matches requirements.txt.

"python-dotenv==1.2.4",
]

Expand Down
8 changes: 0 additions & 8 deletions pydantic-ai/requirements.txt
View file
Open in desktop

This file was deleted.

Loading