Skip to content

docs: remove stale hard-coded Jest test count from Agents.md#41

Merged
tiye merged 2 commits intoperffrom
copilot/sub-pr-40
Mar 14, 2026
Merged

docs: remove stale hard-coded Jest test count from Agents.md#41
tiye merged 2 commits intoperffrom
copilot/sub-pr-40

Conversation

Copy link

Copilot AI commented Mar 14, 2026

The yarn test comment in Agents.md hard-coded "22 Jest tests", which was already inconsistent with the actual test count (27) mentioned later in the same file.

Change

  • Agents.md: # run all 22 Jest tests# run all Jest tests

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: tiye <449224+tiye@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on "Perf by vibe" PR docs: remove stale hard-coded Jest test count from Agents.md Mar 14, 2026
Copilot AI requested a review from tiye March 14, 2026 04:15
Copilot finished work on behalf of tiye March 14, 2026 04:15
@tiye tiye marked this pull request as ready for review March 14, 2026 04:18
Copilot AI review requested due to automatic review settings March 14, 2026 04:18
@tiye tiye merged commit ee34281 into perf Mar 14, 2026
1 check passed
@tiye tiye deleted the copilot/sub-pr-40 branch March 14, 2026 04:18
Copy link

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

Removes a stale, hard-coded Jest test count in the contributor quick commands so the documentation stays accurate as tests are added/removed over time.

Changes:

  • Update yarn test comment from “run all 22 Jest tests” to “run all Jest tests” to avoid an incorrect fixed number.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants