Limit line length in framework/core to 120. - #1554
asuessenbach wants to merge 2 commits into
Conversation
|
As usual :( the Clang Format Check is failing. |
|
Yeah, that Clanf-Format thing is infuriating :( Looking at the CI-setup it's using clang-tools:15.0.0, which has been released in 2022 and is a lot older than what's shipping with recent Visual Studio versions. We should update CI to a version that matches what we use in VS. Which sadly also isn't latest (2026 seems to be clang-tools 19). As a stop-gap you could install the same version CI uses via pip (python) and point Visual Studio to that. |
Switched from 15.0.7 down to 15.0.0. No effect. |
082f7f7 to
f9ede2c
Compare
f9ede2c to
8af8d46
Compare
8af8d46 to
1390423
Compare
|
Rebased, but still failing on Clang Format Check. |
Description
Next part in limiting line length, now framework/core.
Build tested on Win11 with VS2022. Run tested on Win11 with NVidia GPU.
General Checklist:
Please ensure the following points are checked:
Note: The Samples CI runs a number of checks including:
If this PR contains framework changes:
batchcommand line argument to make sure all samples still work properlySample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist: