Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add clang-format shell script #1448

Merged
merged 2 commits into from
May 7, 2023

Conversation

lqxhub
Copy link
Collaborator

@lqxhub lqxhub commented May 6, 2023

加了自动格式化代码的脚本,原来想把格式化的代码加入到 .git/hooks/pre-commit 中, 在实际操作中,

  1. 发现自动添加的功能不好实现
  2. 如果用户电脑上没有 clang-format程序,会导致提交报错

现在在脚本的注释中有添加到 pre-commit的方法,可以自己手动添加

format_code.sh Outdated Show resolved Hide resolved
@wanghenshui wanghenshui self-requested a review May 7, 2023 02:15
@wanghenshui wanghenshui merged commit 8c0be03 into OpenAtomFoundation:unstable May 7, 2023
7 checks passed
@lqxhub lqxhub deleted the format_code branch May 7, 2023 06:26
@yaoyinnan yaoyinnan changed the title add clang-format shell script feat: add clang-format shell script Jul 25, 2023
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
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.

None yet

3 participants