Skip to content

docs: update AGENTS local testing workflows#365

Open
shaypal5 wants to merge 3 commits intomasterfrom
codex/update-agents-md-workflows
Open

docs: update AGENTS local testing workflows#365
shaypal5 wants to merge 3 commits intomasterfrom
codex/update-agents-md-workflows

Conversation

@shaypal5
Copy link
Member

Summary

  • document additional local-testing make targets already present in Makefile
  • document parallel test options supported by scripts/test-local.sh
  • add CACHIER_TEST_CORES environment-variable workflow example

Why

Keep AGENTS.md aligned with actual repo workflows/commands so automation and contributors use the same entry points.

Changes

  • add make test-mongo-inmemory
  • add make test-mongo-also-local
  • add make services-logs
  • add -p/--parallel and -w/--workers options
  • add CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh example

@shaypal5 shaypal5 added the codex label Mar 13, 2026
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.94%. Comparing base (5c4f121) to head (e4e1d9b).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #365   +/-   ##
=======================================
  Coverage   99.94%   99.94%           
=======================================
  Files          12       12           
  Lines        1731     1731           
  Branches      216      216           
=======================================
  Hits         1730     1730           
  Partials        1        1           
Flag Coverage Δ
local 59.21% <ø> (ø)
mongodb 40.55% <ø> (ø)
postgres 43.44% <ø> (ø)
redis 46.67% <ø> (ø)
s3 42.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c4f121...e4e1d9b. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates contributor/agent documentation in AGENTS.md to better reflect existing local-testing workflows (Make targets and scripts/test-local.sh options) used across Cachier’s multi-backend test matrix.

Changes:

  • Document additional make targets (test-mongo-inmemory, test-mongo-also-local, services-logs).
  • Document scripts/test-local.sh parallel options (-p/--parallel, -w/--workers) and CACHIER_TEST_CORES env var usage.

You can also share your feedback on Copilot code review. Take the survey.

@shaypal5 shaypal5 force-pushed the codex/update-agents-md-workflows branch from 930d569 to c36971e Compare March 14, 2026 21:36
@shaypal5
Copy link
Member Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Contributor

Copilot AI commented Mar 14, 2026

@shaypal5 I've opened a new pull request, #366, to work on those changes. Once the pull request is ready, I'll request review from you.

…larify MongoDB Docker requirement (#366)

* Initial plan

* docs: fix broken Makefile mongo targets and clarify MongoDB Docker requirement in AGENTS.md

Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates contributor/agent documentation for local testing so it matches the repo’s current Makefile targets and the unified scripts/test-local.sh workflow.

Changes:

  • Switch MongoDB Makefile targets from the removed scripts/test-mongo-local.sh to scripts/test-local.sh.
  • Document additional Makefile targets (test-mongo-inmemory, test-mongo-also-local, services-logs).
  • Document scripts/test-local.sh parallel options (-p/--parallel, -w/--workers) and the CACHIER_TEST_CORES workflow.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
Makefile Routes MongoDB Docker/local testing targets through the unified scripts/test-local.sh.
AGENTS.md Expands local-testing docs to include additional Make targets, parallel options, and CACHIER_TEST_CORES usage.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants