Skip to content

Refactored sweep code out of load, and added some tests#48

Draft
mccaffers wants to merge 1 commit into
mainfrom
47-feat-extract-sweep-logic-out-of-the-load-command
Draft

Refactored sweep code out of load, and added some tests#48
mccaffers wants to merge 1 commit into
mainfrom
47-feat-extract-sweep-logic-out-of-the-load-command

Refactored sweep code out of load, and added some tests

188e1a8
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 10, 2026 in 2m 5s

Quality Gate failed

Annotations

Check warning on line 23 in source/utilities/decimalConvert.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define and throw a dedicated exception instead of using a generic one.

See more on https://sonarcloud.io/project/issues?id=mccaffers_backtesting-engine-cpp&issues=AZ6wgSowlkGDfR-JdOaH&open=AZ6wgSowlkGDfR-JdOaH&pullRequest=48

Check warning on line 28 in source/utilities/decimalConvert.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define and throw a dedicated exception instead of using a generic one.

See more on https://sonarcloud.io/project/issues?id=mccaffers_backtesting-engine-cpp&issues=AZ6wgSowlkGDfR-JdOaI&open=AZ6wgSowlkGDfR-JdOaI&pullRequest=48

Check warning on line 14 in source/sweep/randomStrategySweep.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code.

See more on https://sonarcloud.io/project/issues?id=mccaffers_backtesting-engine-cpp&issues=AZ6wgSr8lkGDfR-JdOaJ&open=AZ6wgSr8lkGDfR-JdOaJ&pullRequest=48

Check warning on line 27 in source/utilities/decimalConvert.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the init-statement to declare "parsed" inside the if statement.

See more on https://sonarcloud.io/project/issues?id=mccaffers_backtesting-engine-cpp&issues=AZ6wgSowlkGDfR-JdOaG&open=AZ6wgSowlkGDfR-JdOaG&pullRequest=48