Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add test cases for JSON resource marshaling #2532

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

sayalideshmukh10
Copy link
Contributor

@sayalideshmukh10 sayalideshmukh10 commented Oct 26, 2022

Added test cases for JSON resource marshaling - github/teams_discussions.go (file)

  1. TeamDiscussion
    contribution to add tests for resource JSON marshalling #55

Kindly add a Hacktoberfest label for the same

@codecov
Copy link

codecov bot commented Oct 26, 2022

Codecov Report

Merging #2532 (37de95a) into master (9b60dfb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2532   +/-   ##
=======================================
  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.

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @sayalideshmukh10, for updating an old test case! It is very much appreciated!
LGTM.
Merging.

@gmlewis gmlewis merged commit fc230a2 into google:master Oct 26, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants