Skip to content

Re-Formatting#171

Open
ape33 wants to merge 3 commits intodevfrom
formatting
Open

Re-Formatting#171
ape33 wants to merge 3 commits intodevfrom
formatting

Conversation

@ape33
Copy link
Member

@ape33 ape33 commented Jul 28, 2025

Re-formatting of the whole codebase using clang-format-20.1.3

@ape33 ape33 requested a review from 97gamjak July 28, 2025 09:19
@ape33 ape33 added the internal Issues relavant only for developers label Jul 28, 2025
@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 74.57098% with 163 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.29%. Comparing base (6dfa527) to head (e5c9e9d).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
src/input/inputFileParser/hybridInputParser.cpp 0.00% 19 Missing ⚠️
src/input/topologyFileReader/jCouplingSection.cpp 0.00% 12 Missing ⚠️
src/input/restartFileReader/noseHooverSection.cpp 0.00% 8 Missing ⚠️
src/setup/hybridSetup.cpp 0.00% 7 Missing ⚠️
...c/input/inputFileParser/convergenceInputParser.cpp 86.84% 5 Missing ⚠️
...input/parameterFileReader/nonCoulombicsSection.cpp 81.48% 5 Missing ⚠️
src/input/inputFileParser/QMInputParser.cpp 84.00% 4 Missing ⚠️
src/input/inputFileReader.cpp 50.00% 4 Missing ⚠️
src/input/mShakeReader.cpp 75.00% 4 Missing ⚠️
src/input/moldescriptorReader.cpp 78.94% 4 Missing ⚠️
... and 42 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #171      +/-   ##
==========================================
+ Coverage   74.03%   74.29%   +0.25%     
==========================================
  Files         328      328              
  Lines       10730    10895     +165     
  Branches     3376     3385       +9     
==========================================
+ Hits         7944     8094     +150     
- Misses       2749     2764      +15     
  Partials       37       37              

☔ 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.


virtual std::string keyword() = 0;
virtual void processSection(pq::strings &lineElements, pq::Engine &) = 0;
virtual void processSection(
Copy link
Collaborator

Choose a reason for hiding this comment

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

reformat this one

@ape33
Copy link
Member Author

ape33 commented Oct 9, 2025

short for loops not on one line
pointer operator should not be on the next line if its too long

@ape33
Copy link
Member Author

ape33 commented Oct 10, 2025

issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Issues relavant only for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants