Skip to content

Fix tests/README.md CI section to match actual GitHub Actions workflow#351

Merged
shaypal5 merged 1 commit intotesting-improvementsfrom
copilot/sub-pr-349
Feb 28, 2026
Merged

Fix tests/README.md CI section to match actual GitHub Actions workflow#351
shaypal5 merged 1 commit intotesting-improvementsfrom
copilot/sub-pr-349

Conversation

Copy link
Contributor

Copilot AI commented Feb 28, 2026

The CI/CD Integration section in tests/README.md described commands that didn't match the actual .github/workflows/ci-test.yml — wrong marker expressions, incorrect use of pytest-xdist, and a missing s3 backend.

Changes

  • Corrected local test marker: "memory or pickle" -n auto"not mongo and not sql and not redis and not s3" (matches actual workflow)
  • Added s3 backend entry which was missing entirely
  • Clarified execution model: each backend runs in an isolated matrix job, not sequentially in one job
  • Added note that local tests do not use pytest-xdist (-n) in CI

💡 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.

Copilot AI changed the title [WIP] Update parallel testing based on feedback for local testing Fix tests/README.md CI section to match actual GitHub Actions workflow Feb 28, 2026
@shaypal5 shaypal5 marked this pull request as ready for review February 28, 2026 21:22
…flow

Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>
@shaypal5 shaypal5 merged commit db686f3 into testing-improvements Feb 28, 2026
1 check was pending
@shaypal5 shaypal5 deleted the copilot/sub-pr-349 branch February 28, 2026 21:23
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