Skip to content

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

Merged
mwbrooks merged 2 commits into
mainfrom
mwbrooks-remove-requirements-txt
Oct 19, 2026
Merged

mwbrooks merged 2 commits into
mainfrom
mwbrooks-remove-requirements-txt

Conversation

@mwbrooks

Copy link
Copy Markdown
Member

Summary

This pull request removes requirements.txt so pyproject.toml is the single source of dependencies.

Testing

slack create my-agent --template slack-samples/bolt-python-assistant-template --branch mwbrooks-remove-requirements-txt
cd my-agent
slack run
  • Confirm slack create prints "Installed dependencies from pyproject.toml" and the app starts

pyproject.toml already declares every dependency. Removing
requirements.txt leaves one source of dependencies, matching
bolt-python-starter-template#71. CI and the README now install with
`pip install -e ".[dev]"`, and the LLM vendor note moves into
pyproject.toml.
@mwbrooks
mwbrooks requested a review from a team as a code owner October 19, 2026 17:05
@mwbrooks mwbrooks self-assigned this Oct 19, 2026
@mwbrooks
mwbrooks enabled auto-merge (squash) October 19, 2026 17:06
@mwbrooks mwbrooks added update PR only - an update or maintenance is being made to an existing app dependencies Pull requests that update a dependency file labels Oct 19, 2026
# Conflicts:
#	.github/workflows/ci.yml

@WilliamBergamin WilliamBergamin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice 💯

@mwbrooks
mwbrooks merged commit c3ec1cf into main Oct 19, 2026
3 checks passed
@mwbrooks
mwbrooks deleted the mwbrooks-remove-requirements-txt branch October 19, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file update PR only - an update or maintenance is being made to an existing app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants