Skip to content

Conversation

monadbobo
Copy link
Contributor

@monadbobo monadbobo commented Jul 2, 2019

Nebula code is to use the Google C++ Style,
so added clang-format config to automatically format the code.

close #738

@monadbobo monadbobo force-pushed the clang_format_config branch from 0e91baf to 379b8b9 Compare August 8, 2019 08:05
@monadbobo
Copy link
Contributor Author

close #738

Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

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

LGTM.

@dutor dutor added the ready-for-testing PR: ready for the CI test label Sep 23, 2019
@nebula-community-bot
Copy link
Member

Unit testing passed.

@dutor dutor merged commit 3c306da into vesoft-inc:master Sep 23, 2019
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
* add a clang-format config.

* Remove AlignConsecutiveAssignments option for clang-format.
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Mar 21, 2022
* fix meta http replace host handler

* fix http replace test and add zone test

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

Co-authored-by: pengwei.song <90180021+pengweisong@users.noreply.github.com>
Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

We should use the clang-format to format the code.
4 participants