Add test cases for JSON resource marshaling#1975
Conversation
- InteractionRestriction
google-cla
Bot
added
the
cla: yes
Codecov Report
@@ Coverage Diff @@
## master #1975 +/- ##
=======================================
Coverage 97.84% 97.84%
=======================================
Files 105 105
Lines 6809 6809
=======================================
Hits 6662 6662
Misses 80 80
Partials 67 67 Continue to review full report at Codecov.
|
gmlewis
left a comment
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @sagar23sj !
One minor tweak to fix, please, and then it LGTM.
Whenever a new file is created, we use the year it was created as its copyright year.
| @@ -0,0 +1,26 @@ | |||
| // Copyright 2018 The go-github AUTHORS. All rights reserved. | |||
There was a problem hiding this comment.
| // Copyright 2018 The go-github AUTHORS. All rights reserved. | |
| // Copyright 2021 The go-github AUTHORS. All rights reserved. |
There was a problem hiding this comment.
Sure @gmlewis. Will make the change.
Thank you for your instantaneous feedback.
gmlewis
left a comment
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @sagar23sj !
LGTM.
Awaiting second LGTM before merging.
3 participants
Resources Covered :
Helps with issue: #55