Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 17, 2025

Important

Update test mode from Jest to Vitest in .roomodes, including role definitions, file regex, and custom instructions.

  • Behavior:
    • Update test mode from Jest to Vitest in .roomodes.
    • Change role definition to focus on Vitest expertise.
    • Update file regex to include .spec files and Vitest config files.
  • Instructions:
    • Modify custom instructions to reflect Vitest usage, including test running requirements and function imports.
  • Misc:
    • Update whenToUse description for test mode.

This description was created by Ellipsis for 1151625. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners June 17, 2025 12:09
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 17, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 17, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

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

Looks good! I like that Sonnet 4 was encouraged to take a look at other tests before creating one.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 17, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Jun 17, 2025
@hannesrudolph hannesrudolph added PR - Needs Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 17, 2025
@mrubens mrubens merged commit 3e2aec2 into main Jun 17, 2025
25 checks passed
@mrubens mrubens deleted the update_test_mode branch June 17, 2025 15:55
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 17, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Jun 17, 2025
valekseev pushed a commit to valekseev/Roo-Code that referenced this pull request Jun 18, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Needs Review size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants