-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add test case for JSON resource marshaling #2541
Add test case for JSON resource marshaling #2541
Conversation
…to Team_Test_ExternalGroupMember
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @rahulkarmore !
LGTM.
Waiting for GitHub Actions to perform all checks before merging.
@rahulkarmore - if GitHub Actions doesn't kick in within an hour or so, please feel free to tweak one of your constants in your PR and push the change to this PR... hopefully that will kick GHA into gear... it looks like it is currently either having issues or is just sleeping. 😁 |
Thank you, @rahulkarmore ! Do you want to resolve the conflicts by merging in the master branch (again), or would you like me to do that? |
Codecov Report
@@ Coverage Diff @@
## master #2541 +/- ##
=======================================
Coverage 97.97% 97.97%
=======================================
Files 123 123
Lines 10813 10813
=======================================
Hits 10594 10594
Misses 150 150
Partials 69 69 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…to Team_Test_ExternalGroupMember
92a2236
to
cb9253a
Compare
resolved conflict |
Hi creator,
Added test cases for JSON Resources :
ExternalGroup
Helps: #55
I have followed all the guidelines to contribute to this repository.
Please revert in case of any changes are needed.
I am a first-time contributor to Hackoctober, It will be a great help if u can encourage me & accept my contribution.