Skip to content

fix prettier formatting from old PRs #6391

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Jul 2, 2025

Some more old PRs were merged that were passing tests BEFORE we added the prettier CICD rule


Summary by cubic

Fixed code formatting issues in several files to match the current Prettier rules. This cleans up formatting from older PRs merged before the Prettier check was added.

@RomneyDa RomneyDa requested a review from a team as a code owner July 2, 2025 00:33
@RomneyDa RomneyDa requested review from tomasz-stefaniak and removed request for a team July 2, 2025 00:33
Copy link

netlify bot commented Jul 2, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 2364bba
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68648fbf11bd0e0008fc636c

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 2, 2025
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 1 issue across 6 files. Review it in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

@@ -39,7 +39,7 @@ type NextEditModelName = "mercury-coder-nextedit" | "this field is not used";
const NEXT_EDIT_MODEL_TEMPLATES: Record<NextEditModelName, NextEditTemplate> = {
"mercury-coder-nextedit": {
template:
"### User Edits:\n\n{{{userEdits}}}\n\n### User Excerpts:\n\n\`\`\`{{{languageShorthand}}}\n{{{userExcerpts}}}\`\`\`",
"### User Edits:\n\n{{{userEdits}}}\n\n### User Excerpts:\n\n```{{{languageShorthand}}}\n{{{userExcerpts}}}```",
Copy link

Choose a reason for hiding this comment

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

Using triple backticks (```) for code blocks in a template string may cause issues if the userExcerpts or languageShorthand variables themselves contain triple backticks, potentially breaking the formatting or causing injection vulnerabilities. Consider using a more robust escaping or delimiter strategy.

@RomneyDa
Copy link
Collaborator Author

RomneyDa commented Jul 2, 2025

seems like working on main now

@RomneyDa RomneyDa closed this Jul 2, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Jul 2, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant