Skip to content

CI流程#34

Open
yousongyang wants to merge 1 commit into
owent:mainfrom
yousongyang:main
Open

CI流程#34
yousongyang wants to merge 1 commit into
owent:mainfrom
yousongyang:main

Conversation

@yousongyang

Copy link
Copy Markdown
Contributor

No description provided.

push:
branches:
- main
- master

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

不需要master,这个分支仅仅是归档了,不需要再分析了

push:
branches:
- main
- master

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

同上

Comment thread ci/install_iwyu.sh
echo "Configuring include-what-you-use..."
cmake .. \
-G Ninja \
-DCMAKE_BUILD_TYPE=Release \

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

和上面不匹配,用 RelWithDebInfo

Comment thread cmake_dev.sh
;;
t)
CMAKE_CLANG_TIDY="-D -checks=* --";
CMAKE_CLANG_TIDY="-DCMAKE_CXX_CLANG_TIDY=clang-tidy";

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

这个原本是传给clang-tidy的选项,如果不需要且是换成cmake选项,用 CMAKE_OPTIONS="$CMAKE_OPTIONS -DCMAKE_CXX_CLANG_TIDY=$(which clang-tidy)" 即可

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.

2 participants