Skip to content

chore(global): global config formatting - #3616

Open
MaoMengww wants to merge 1 commit into
apache:developfrom
MaoMengww:chore(global)/global-config-formatting
Open

chore(global): global config formatting#3616
MaoMengww wants to merge 1 commit into
apache:developfrom
MaoMengww:chore(global)/global-config-formatting

Conversation

@MaoMengww

Copy link
Copy Markdown

Description

Fixes # (issue)
For this PR, I consider the field ordering of the config structs already reasonable and left it unchanged. Changes made:

  1. Rewrote incomplete sentences into complete sentences. For example, // MaxServerRecvMsgSize max size of server receive message was changed to // MaxServerRecvMsgSize defines the max size of server receive message.

  2. Unified the type-comment style to // XConfig represents. For example, // TripleConfig represents the config of triple protocol.

  3. Supplemented some missing comments.

  4. Added periods to complete sentences.

  5. Adjusted some blank lines to make the grouping more obvious.

Refs #3598 (task19)

Checklist

  • I confirm the target branch is develop
  • Code has passed local testing
  • I have added tests that prove my fix is effective or that my feature works

@sonarqubecloud

sonarqubecloud Bot commented Aug 8, 2026

Copy link
Copy Markdown

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.89%. Comparing base (60d1c2a) to head (37f15d2).
⚠️ Report is 896 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3616      +/-   ##
===========================================
+ Coverage    46.76%   53.89%   +7.12%     
===========================================
  Files          295      474     +179     
  Lines        17172    36388   +19216     
===========================================
+ Hits          8031    19610   +11579     
- Misses        8287    15204    +6917     
- Partials       854     1574     +720     

☔ View full report in Codecov by Harness.
📢 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.

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