Skip to content

Fix Prettier formatting violations#344

Merged
saint2706 merged 2 commits intomainfrom
copilot/lint-and-format-code
Feb 12, 2026
Merged

Fix Prettier formatting violations#344
saint2706 merged 2 commits intomainfrom
copilot/lint-and-format-code

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

Fixed code style violations in src/components/PythonRunner.tsx to comply with Prettier configuration.

Changes

  • Reformatted useImperativeHandle hook call with proper line breaks
  • Collapsed single-line JSX text content per style guide
- useImperativeHandle(ref, () => ({
-   run: handleRun,
- }), [handleRun])
+ useImperativeHandle(
+   ref,
+   () => ({
+     run: handleRun,
+   }),
+   [handleRun],
+ )

All linting, formatting, type checking, and build now pass without errors or warnings.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: saint2706 <45678566+saint2706@users.noreply.github.com>
Copilot AI changed the title [WIP] Lint and format code to fix all errors and warnings Fix Prettier formatting violations Feb 12, 2026
Copilot AI requested a review from saint2706 February 12, 2026 14:24
@saint2706 saint2706 marked this pull request as ready for review February 12, 2026 14:24
Copilot AI review requested due to automatic review settings February 12, 2026 14:24
@saint2706 saint2706 merged commit 6ad10b4 into main Feb 12, 2026
@saint2706 saint2706 deleted the copilot/lint-and-format-code branch February 12, 2026 14:25
Copy link
Contributor

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.

Pull request overview

This PR updates PythonRunner to resolve Prettier formatting violations, primarily adjusting hook call formatting and JSX whitespace/layout.

Changes:

  • Reformatted the useImperativeHandle call to match Prettier’s multiline style.
  • Collapsed the empty-output <pre> JSX content to a single line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 102 to 104
{output === '' && !error && (
<pre className="python-runner__stdout python-runner__stdout--empty">
(no output)
</pre>
<pre className="python-runner__stdout python-runner__stdout--empty">(no output)</pre>
)}
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

This JSX change can alter rendered output: inside a

 (and with white-space: pre-wrap in CSS), newline/indentation whitespace is significant. Collapsing to a single line removes the leading/trailing whitespace that the previous multi-line markup would have produced. If this PR is meant to be style-only, consider rendering the text in a way that preserves the exact previous whitespace, or update the PR description to note the user-visible behavior change.

Copilot uses AI. Check for mistakes.
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.

2 participants