Skip to content

chore: Add cpp binding Google style clang-format && format the code#3581

Merged
Xuanwo merged 1 commit intoapache:mainfrom
JackDrogon:feature/format_cpp_binding
Nov 14, 2023
Merged

chore: Add cpp binding Google style clang-format && format the code#3581
Xuanwo merged 1 commit intoapache:mainfrom
JackDrogon:feature/format_cpp_binding

Conversation

@JackDrogon
Copy link
Copy Markdown
Contributor

Add cpp binding Google style clang-format && format the code

Signed-off-by: Jack Drogon <jack.xsuperman@gmail.com>
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Pleae also update CMake for code format job

Comment thread bindings/cpp/.clang-format
@Xuanwo Xuanwo changed the title Add cpp binding Google style clang-format && format the code chore: Add cpp binding Google style clang-format && format the code Nov 14, 2023
@Xuanwo Xuanwo requested a review from silver-ymz November 14, 2023 06:02
@github-actions github-actions Bot added the releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell label Nov 14, 2023
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Nov 14, 2023

Hi, @silver-ymz, would you like to take a review?

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

I don't like git pre-commit. It's ok for me to merge this PR as is if @silver-ymz is ok with the Google Code Style.

Copy link
Copy Markdown
Member

@silver-ymz silver-ymz left a comment

Choose a reason for hiding this comment

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

Thanks

@Xuanwo Xuanwo merged commit 6d46e38 into apache:main Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants