Skip to content

fix: replace non-existent test-mongo-local.sh calls in Makefile and clarify MongoDB Docker requirement#366

Merged
shaypal5 merged 2 commits intocodex/update-agents-md-workflowsfrom
copilot/sub-pr-365
Mar 14, 2026
Merged

fix: replace non-existent test-mongo-local.sh calls in Makefile and clarify MongoDB Docker requirement#366
shaypal5 merged 2 commits intocodex/update-agents-md-workflowsfrom
copilot/sub-pr-365

Conversation

Copy link
Contributor

Copilot AI commented Mar 14, 2026

make test-mongo-local, make test-mongo-docker, and make test-mongo-also-local were silently broken — all three invoked ./scripts/test-mongo-local.sh which doesn't exist in the repo.

Changes

  • Makefile: Replace broken ./scripts/test-mongo-local.sh calls with the existing ./scripts/test-local.sh:

    • test-mongo-docker / test-mongo-local./scripts/test-local.sh mongo
    • test-mongo-also-local./scripts/test-local.sh mongo memory pickle
  • AGENTS.md: Correct the blanket "external backends require Docker" note — MongoDB runs in-memory by default (make test-mongo-inmemory), Docker is only required for Redis and SQL.


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

…quirement in AGENTS.md

Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>
Copilot AI changed the title [WIP] Update AGENTS local testing workflows fix: replace non-existent test-mongo-local.sh calls in Makefile and clarify MongoDB Docker requirement Mar 14, 2026
Copilot AI requested a review from shaypal5 March 14, 2026 21:39
@shaypal5 shaypal5 marked this pull request as ready for review March 14, 2026 21:57
@shaypal5 shaypal5 merged commit 7165a1f into codex/update-agents-md-workflows Mar 14, 2026
1 check passed
@shaypal5 shaypal5 deleted the copilot/sub-pr-365 branch March 14, 2026 21:57
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