Conversation
|
✅ All contributors have signed the CLA |
|
I have read the CLA Document and I hereby sign the CLA |
|
recheck |
Unit Tests Summary 1 files 58 suites 3m 14s ⏱️ Results for commit 9652ade. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit 8924f92 ♻️ This comment has been updated with latest results. |
clarkliming
left a comment
There was a problem hiding this comment.
I think the current version is corret, using n as denominator. But I would like also include @tianfang1121 to have a look
BFalquet
left a comment
There was a problem hiding this comment.
Please update tern version requirement in DESCRIPTION
Code Coverage SummaryDiff against mainResults for commit: 9652ade Minimum allowed coverage is ♻️ This comment has been updated with latest results |
The way the percentage is calculated right now matches with the analysis specification in GDSR LBT14 template |
clarkliming
left a comment
There was a problem hiding this comment.
thank you, looks good
LBT14 snapshot updates following insightsengineering/tern#1326