Closed ccascone closed 3 years ago
Merging #215 (14fe521) into master (2152bee) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #215 +/- ##
=======================================
Coverage 83.39% 83.39%
=======================================
Files 13 13
Lines 572 572
Branches 65 65
=======================================
Hits 477 477
Misses 95 95
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 2152bee...14fe521. Read the comment docs.
Good catch @hwchiu, I'll see if I can add a json
validation to our testing
I still don't see this group in GitHub. When should I be able to?
On Tue, Feb 9, 2021 at 11:05 PM Hung-Wei Chiu notifications@github.com wrote:
@hwchiu approved this pull request.
LGTM
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OpenNetworkingFoundation/cla-manager/pull/215#pullrequestreview-587280596, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANU5VDKAHTKN3IAZDGTJ4DLS6IO3BANCNFSM4XMISW2Q .
Ain
@aindermitte PR merged. The group should be synced in the next hour.
As requested by @aindermitte.