FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove redundant requirements.txt in favor of pyproject.toml by Copilot · Pull Request #56 · slack-samples/bolt-python-assistant-template · GitHub

Remove redundant requirements.txt in favor of pyproject.toml - #56

Draft
zimeg with Copilot wants to merge 3 commits into
mainfrom
copilot/remove-requirements-file
Draft

Remove redundant requirements.txt in favor of pyproject.toml#56
zimeg with Copilot wants to merge 3 commits into
mainfrom
copilot/remove-requirements-file

Conversation

Copilot AI commented Mar 2, 2026
edited
Loading

Copy link
Copy Markdown
  • Explore repository structure and understand the issue
  • Delete requirements.txt (redundant since pyproject.toml manages dependencies)
  • Update README.md install command: pip install ".[dev]" (installs all runtime + dev deps without requiring editable mode)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: zimeg <18134219+zimeg@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove requirements.txt file from repository Remove redundant requirements.txt in favor of pyproject.toml Mar 2, 2026
Copilot finished work on behalf of zimeg March 2, 2026 20:40

zimeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

🤖

Comment thread README.md Outdated

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@copilot Does this include all dependencies? Or just develeopment?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@copilot Also What does this mean?

  ERROR: File "setup.py" or "setup.cfg" not found. Directory cannot be installed in editable mode: /Users/user/programming/tools/slack-cli/my-app
  (A "pyproject.toml" file was found, but editable mode currently requires a setuptools-based build.)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Good catches! pip install ".[dev]" installs everything — all runtime dependencies from [project].dependencies (slack-sdk, slack-bolt, slack-cli-hooks, openai, python-dotenv) plus the dev tools from [project.optional-dependencies].dev (pytest, ruff).

The -e editable flag was the problem. The pyproject.toml has no [build-system] table, so pip doesn't know which build backend to use for editable installs, causing that error. Dropped the flag in 164897a — pip install ".[dev]" works correctly without editable mode, which is fine for an app template where you run the code rather than develop the package itself.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

That error was caused by the -e (editable mode) flag — pip needs a [build-system] section in pyproject.toml (or a setup.py/setup.cfg) to perform editable installs, and this project has neither. Fixed in 164897a by removing the -e flag: pip install ".[dev]" works without requiring a build backend.

Co-authored-by: zimeg <18134219+zimeg@users.noreply.github.com>
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL