Add feedback from future devs

[holgstr]

Fixes #105

[github-actions]

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  ---------
R/assertions.R                  15       0  100.00%
R/corPFSOS.R                    96      20  79.17%   317-336
R/empSignificant.R              59       0  100.00%
R/estimateParams.R              80       0  100.00%
R/eventTracking.R               88       2  97.73%   48, 244
R/getClinicalTrials.R           71       0  100.00%
R/getSimulatedData.R           109       0  100.00%
R/getWaitTimeSum.R              29       2  93.10%   45, 48
R/hazardFunctions.R             26       0  100.00%
R/piecewiseDistribution.R       32       0  100.00%
R/piecewiseHazards.R            18       0  100.00%
R/survivalFunctions.R          157       0  100.00%
R/transitionParameters.R        42       0  100.00%
TOTAL                          822      24  97.08%

Diff against main

Filename        Stmts    Miss  Cover
------------  -------  ------  -------
R/corPFSOS.R       +1      +1  -0.83%
TOTAL              +1      +1  -0.12%

Results for commit: fab5202

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

  1 files   16 suites   2m 14s ⏱️
104 tests 103 ✅ 1 💤 0 ❌
230 runs  229 ✅ 1 💤 0 ❌

Results for commit fab5202.

♻️ This comment has been updated with latest results.

[danielinteractive]

Thanks @holgstr - is it worth a bug fix NEWS entry?

[holgstr]

@danielinteractive yes :)

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
getSimulatedDataDistib	💔	$96.47$	$+2.81$	$0$	$0$	$0$	$0$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
getSimulatedDataDistib	💔	$34.39$	$+1.47$	getSimulatedData_generates_distributions_as_expected_PW
getSimulatedDataDistib	💔	$50.53$	$+1.25$	getSimulatedData_generates_distributions_as_expected_Weibull

Results for commit 0935406

♻️ This comment has been updated with latest results.

[danielinteractive]

Thanks @holgstr , one minor detail and then good to merge