Chris Hennes

`communicate()` has to be called after a final `kill()` to get the output. Also adds unit tests for the function. Related to #18380.

CI: Remove readability-identifier-length check

3

As discussed in the PR Review Meeting this morning, this removes the `readability-identifier-length` automated check, which creates more noise than it is worth. We plan to instead rely on code...

AddonManager: main class is too complex and needs refractoring

1

### Is there an existing issue for this? - [X] I have searched the existing issues ### Forums discussion _No response_ ### Version 0.21 (Development) ### Full version info ```shell...

Addon Manager: refactor process_string_to_datetime

11

Move it to the utilities file and get it under test.

[Problem] Images created from BitmapFactory are not themeable

3

### Is there an existing issue for this? - [X] I have searched the existing issues ### Version 0.21 (Development) ### Full version info ```shell OS: Windows 11 build 22621...

cMake: No project-level formatting standard

1

### Is there an existing issue for this? - [X] I have searched the existing issues ### Problem description Our cMake files use a variety of different formatting standards, and...

The path specified in package.xml for the package's icon does not exist.

Closes https://github.com/FreeCAD/FreeCAD/security/code-scanning/5306 by completely removing the code block, which was no longer needed.

Right now if we include 3rd party code in the analysis there are nearly 10,000 warnings. This isn't productive, so reduce the analysis to only our code (and also exclude...

There is a malformed method in PartFeatures.h that was leftover from the TNP merge: it didn't matter that it was malformed because in the end we didn't need any of...