Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 9, 2025

Annoying how this keeps getting out of sync...


Important

Run Prettier on src directory to ensure consistent code formatting across multiple files.

  • Formatting:
    • Run Prettier on src/core/prompts/sections/__tests__/objective.test.ts, src/core/prompts/sections/__tests__/tool-use-guidelines.test.ts, and src/core/prompts/sections/objective.ts to adjust whitespace and line breaks.
    • Apply Prettier formatting to src/core/prompts/sections/rules.ts, src/core/prompts/sections/tool-use-guidelines.ts, and src/core/prompts/tools/attempt-completion.ts for consistent indentation and line breaks.
    • Format src/core/tools/__tests__/attemptCompletionTool.experiment.test.ts with Prettier to ensure consistent code style.

This description was created by Ellipsis for 85c7d36. 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 9, 2025 21:08
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 9, 2025
@mrubens mrubens merged commit 1d63fb6 into main Jun 9, 2025
13 checks passed
@mrubens mrubens deleted the run_prettier branch June 9, 2025 21:10
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 9, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 9, 2025
@cte
Copy link
Collaborator

cte commented Jun 9, 2025

Want me to add this to CI and fail the build?

@mrubens
Copy link
Collaborator Author

mrubens commented Jun 9, 2025

Want me to add this to CI and fail the build?

Yeah I think so - it causes a lot of hassle otherwise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants