Skip to content

Conversation

andreasmcdermott
Copy link
Member

Not used much yet, but coming as fast follows.

@andreasmcdermott andreasmcdermott requested a review from Copilot April 9, 2025 22:50
Copy link

@Copilot 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.

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

depth: options.depth !== undefined ? options.depth - 1 : undefined,
});

if (formattedItem.includes("\n")) return `${indent}- \n${formattedItem}`;
Copy link
Preview

Copilot AI Apr 9, 2025

Choose a reason for hiding this comment

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

[nitpick] When an array item contains newlines, the formatting may lead to inconsistent indentation. Consider revising the prefix so that the nested lines are indented consistently with the rest of the output.

Suggested change
if (formattedItem.includes("\n")) return `${indent}- \n${formattedItem}`;
if (formattedItem.includes("\n")) return `${indent}- \n${nextIndent}${formattedItem.replace(/\n/g, `\n${nextIndent}`)}`;

Copilot uses AI. Check for mistakes.

@andreasmcdermott andreasmcdermott merged commit 5c41cef into main Apr 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant