Skip to content

docs: update AGENTS local testing workflows#365

Merged
shaypal5 merged 3 commits into
masterfrom
codex/update-agents-md-workflows
Mar 20, 2026
Merged

docs: update AGENTS local testing workflows#365
shaypal5 merged 3 commits into
masterfrom
codex/update-agents-md-workflows

Conversation

@shaypal5

Copy link
Copy Markdown
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

codecov Bot commented Mar 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.94%. Comparing base (0df7dc4) to head (cf60c98).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #365   +/-   ##
=======================================
  Coverage   99.94%   99.94%           
=======================================
  Files          12       12           
  Lines        1735     1735           
  Branches      218      218           
=======================================
  Hits         1734     1734           
  Partials        1        1           
Flag Coverage Δ
local 59.30% <ø> (ø)
mongodb 40.69% <ø> (ø)
postgres 43.40% <ø> (ø)
redis 46.80% <ø> (ø)
s3 42.47% <ø> (ø)

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 0df7dc4...cf60c98. Read the comment docs.

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread AGENTS.md
Comment thread AGENTS.md Outdated
@shaypal5 shaypal5 force-pushed the codex/update-agents-md-workflows branch from 930d569 to c36971e Compare March 14, 2026 21:36
@shaypal5

Copy link
Copy Markdown
Member Author

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

Copilot AI commented Mar 14, 2026

Copy link
Copy Markdown
Contributor

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread AGENTS.md Outdated
shaypal5 and others added 3 commits March 20, 2026 10:28
…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>
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

Labels

area: agent-workflows Agent instructions, AI-assisted workflow docs, or bot-maintained guidance. area: ci GitHub Actions, pre-commit, codecov, or automation config. area: docs README, examples, contributor docs, or repo documentation. outcome: merged Merged into the repository. size: small Small PR by changed-file count. source: codex Opened from a Codex-assisted workflow. testing and development type: docs Documentation-only or documentation-focused change. type: tests Tests, fixtures, test coverage, or test data.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants