Skip to content

[FORMAT] Pre-Commit Formatting - #172

Merged
mark14wu merged 1 commit into
mainfrom
pre_commit_formatting
Sep 24, 2025
Merged

mark14wu merged 1 commit into
mainfrom
pre_commit_formatting

Conversation

@mark14wu

Copy link
Copy Markdown
Collaborator

Somehow I forgot to format in PR #171.

@mark14wu

Copy link
Copy Markdown
Collaborator Author

@codex review.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

@mark14wu
mark14wu requested a review from Copilot September 24, 2025 13:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 pre-commit formatting changes that were missed in PR #171. The changes focus on code formatting improvements to maintain consistent style.

  • Reformats a multi-line conditional expression for better readability
  • Adds whitespace for improved code structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Jokeren

Jokeren commented Sep 24, 2025

Copy link
Copy Markdown
Member

I thought you should be blocked from merging if not all checks have passed.

@mark14wu

Copy link
Copy Markdown
Collaborator Author

It supposed to be. But somehow it's not.

@mark14wu
mark14wu merged commit f3eafcb into main Sep 24, 2025
1 check passed
@mark14wu
mark14wu deleted the pre_commit_formatting branch September 24, 2025 16:15
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.

3 participants