Skip to content

Migrate from dev-requirements.txt to pyproject.toml for dependency management#78

Merged
simonw merged 3 commits intomainfrom
copilot/update-github-actions-config
Oct 21, 2025
Merged

Migrate from dev-requirements.txt to pyproject.toml for dependency management#78
simonw merged 3 commits intomainfrom
copilot/update-github-actions-config

Conversation

Copy link
Contributor

Copilot AI commented Oct 21, 2025

This PR updates the GitHub Actions workflows and documentation to use pyproject.toml for dependency management instead of the now-removed dev-requirements.txt file.

Changes

GitHub Actions Workflow

Updated .github/workflows/test.yml to:

  • Use pyproject.toml as the pip cache dependency path instead of dev-requirements.txt
  • Install dependencies using pip install -e . which reads from pyproject.toml

Before:

cache-dependency-path: 'dev-requirements.txt'
# ...
run: pip install -r dev-requirements.txt

After:

cache-dependency-path: 'pyproject.toml'
# ...
run: pip install -e .

Documentation

Updated TOOLS_GUIDE.md to reflect the new installation method:

  • Repository structure diagram now shows pyproject.toml instead of dev-requirements.txt
  • Test running instructions updated to use pip install -e .
  • GitHub workflow examples updated to match actual configuration

Build Configuration

Enhanced .gitignore to exclude Python build artifacts:

  • *.egg-info/
  • __pycache__/
  • *.pyc
  • .pytest_cache/

Verification

The pyproject.toml file already contains pytest-playwright as a dependency, so no changes to dependencies were needed. This PR simply aligns the CI/CD pipeline and documentation with the existing project structure.

Fixes the issue where GitHub Actions would fail looking for a non-existent dev-requirements.txt file.

Original prompt

This section details on the original issue you should resolve

<issue_title>I replaced dev-requirements.txt with pyproject.toml and stuff broke</issue_title>
<issue_description>Need to update the GitHub Actions configurations to work with the new pyproject.toml project, install from that instead of using dev-requirements.txt. Also update existing docs that mention dev-requirements.txt.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #77


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: simonw <9599+simonw@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 21, 2025

Deploying tools with  Cloudflare Pages  Cloudflare Pages

Latest commit: aa2e711
Status: ✅  Deploy successful!
Preview URL: https://ea1c0831.tools-b1q.pages.dev
Branch Preview URL: https://copilot-update-github-action.tools-b1q.pages.dev

View logs

…equirements.txt

Co-authored-by: simonw <9599+simonw@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Actions to use pyproject.toml Migrate from dev-requirements.txt to pyproject.toml for dependency management Oct 21, 2025
Copilot AI requested a review from simonw October 21, 2025 17:27
@simonw simonw marked this pull request as ready for review October 21, 2025 19:45
@simonw simonw merged commit 6113051 into main Oct 21, 2025
4 of 6 checks passed
@simonw simonw deleted the copilot/update-github-actions-config branch October 21, 2025 19:46
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.

I replaced dev-requirements.txt with pyproject.toml and stuff broke

2 participants