Skip to content

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

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

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

Conversation

Copilot AI commented Mar 14, 2026

Copy link
Copy Markdown
Contributor

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

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

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