Skip to content

style: add missing brackets and remove unused including headers#914

Open
SYaoJun wants to merge 5 commits intoapache:mainfrom
SYaoJun:brackets_410
Open

style: add missing brackets and remove unused including headers#914
SYaoJun wants to merge 5 commits intoapache:mainfrom
SYaoJun:brackets_410

Conversation

@SYaoJun
Copy link
Copy Markdown
Contributor

@SYaoJun SYaoJun commented Apr 10, 2026

Reason for this PR

fix some style problems, including single line statements without brackets and unused header files.

Signed-off-by: syaojun <libevent@yeah.net>
@SYaoJun SYaoJun marked this pull request as ready for review April 10, 2026 13:40
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 10, 2026

Codecov Report

❌ Patch coverage is 36.36364% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.60%. Comparing base (b21da29) to head (55cb062).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
cpp/src/graphar/high-level/graph_reader.h 12.50% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main     #914   +/-   ##
=========================================
  Coverage     80.60%   80.60%           
  Complexity      615      615           
=========================================
  Files            94       94           
  Lines         10709    10710    +1     
  Branches       1055     1055           
=========================================
+ Hits           8632     8633    +1     
  Misses         1837     1837           
  Partials        240      240           
Flag Coverage Δ
cpp 70.91% <36.36%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

SYaoJun added 4 commits April 10, 2026 22:05
Signed-off-by: syaojun <libevent@yeah.net>
Signed-off-by: syaojun <libevent@yeah.net>
Signed-off-by: syaojun <libevent@yeah.net>
Signed-off-by: syaojun <libevent@yeah.net>
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