Skip to content

Conversation

@jkowalski
Copy link
Contributor

No description provided.

@jkowalski jkowalski requested a review from Copilot May 17, 2025 15:39
Copy link

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 applies automated formatting via Prettier across the codebase, ensuring a more consistent code style and updated CI checks.

  • Consistent use of double quotes and indentation across React components and utility modules.
  • Added Prettier commands to package.json and the CI workflow for code quality.
  • Minor adjustments to multi-line JSX formatting for improved readability.

Reviewed Changes

Copilot reviewed 90 out of 90 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/components/notifications/* Reformatted JSX and function definitions for consistency.
src/components/SnapshotEstimation.jsx Changed multi-line chaining and indentation.
src/components/SetupRepository* All repository setup components reformatted with updated prop formatting.
src/components/Logs.jsx Re-indented and formatted log rendering and helper functions.
src/components/Directory* Updated link and breadcrumb formatting.
src/App.jsx Adjusted formatting in routing and Navbar components.
package.json & .github/workflows/build.yaml Added Prettier commands and updated dependency versions.
Comments suppressed due to low confidence (1)

src/components/SetupRepositoryGCS.jsx:30

  • The 'Object Name Prefix' field is configured with type 'password', which might unintentionally mask input. Verify if the prefix should be treated as sensitive data; if not, remove the type attribute to display the input normally.
{OptionalField(this, "Object Name Prefix", "prefix", { placeholder: "enter object name prefix or leave empty", type: "password" })}

@jkowalski jkowalski merged commit 4383559 into kopia:main May 17, 2025
2 checks passed
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.

1 participant