Open nitinsubz opened 3 years ago
Merging #277 (bea4351) into master (271efd6) will increase coverage by
0.72%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #277 +/- ##
==========================================
+ Coverage 89.77% 90.50% +0.72%
==========================================
Files 9 9
Lines 313 337 +24
Branches 22 25 +3
==========================================
+ Hits 281 305 +24
Misses 32 32
Impacted Files | Coverage Δ | |
---|---|---|
common/model/agreement.js | 92.45% <100.00%> (+1.75%) |
:arrow_up: |
common/model/appUser.js | 96.77% <100.00%> (+0.47%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 271efd6...bea4351. Read the comment docs.
The feature appears to be working fine, but can we add the Signer
of the Agreement and the Company
(if any) to the table?
This PR shows CLA's that a user is considered a contributor as.
At the bottom of the home CLA screen, there now exists a table that lists all the CLA's that the user is covered by.
A few questions:
Fixes #196