Skip to content

Conversation

shenxianpeng
Copy link
Collaborator

Right now merge PR will be added thread-comments, see b20a30f

I would like to disable it for merged commits.

Right now merge PR will be added thread-comments, see b20a30f

I would like to disable it for merged commits.
@shenxianpeng shenxianpeng added the developer Changes which impact developers label Sep 2, 2025
@shenxianpeng shenxianpeng requested a review from a team as a code owner September 2, 2025 19:48
@shenxianpeng shenxianpeng requested review from 2bndy5 and removed request for a team September 2, 2025 19:48
Copy link
Contributor

github-actions bot commented Sep 2, 2025

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format (v16.0.6) reports: 2 file(s) not formatted
  • docs/examples/demo/demo.hpp
  • docs/examples/demo/demo.cpp
clang-tidy (v16.0.6) reports: 7 concern(s)

Have any feedback or feature suggestions? Share it here.

Copy link
Collaborator

@2bndy5 2bndy5 left a comment

Choose a reason for hiding this comment

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

we could also update it to only post comments (and file-annotations) using clang v16

@shenxianpeng shenxianpeng merged commit e94da0a into main Sep 3, 2025
84 checks passed
@shenxianpeng shenxianpeng deleted the shenxianpeng-patch-1 branch September 3, 2025 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer Changes which impact developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants