Skip to content
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 ability to specify 'since' request parameter to issue comments #2008

Merged
merged 3 commits into from
Sep 21, 2019

Conversation

hnrkndrssn
Copy link
Contributor

@hnrkndrssn hnrkndrssn commented Sep 21, 2019

I noticed that IObservable<IssueComment> GetAllForIssue(long repositoryId, int number, ApiOptions otpions) didn't specify AcceptHeaders.ReactionsPreview so I added this as part of my changes.

  • Tests ? (I checked the GetAllForRepository method and noticed that there are no tests that use the IssueCommentRequest parameter, but I can add tests for the GetAllForIssue if desired) Codecov clearly wants me to write some tests 😁

Fixes #1960

@codecov
Copy link

codecov bot commented Sep 21, 2019

Codecov Report

Merging #2008 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2008      +/-   ##
==========================================
+ Coverage   70.82%   70.87%   +0.05%     
==========================================
  Files         529      529              
  Lines       13928    13956      +28     
==========================================
+ Hits         9864     9892      +28     
  Misses       4064     4064
Impacted Files Coverage Δ
Octokit/Clients/IssueCommentsClient.cs 91.42% <100%> (+2.14%) ⬆️
....Reactive/Clients/ObservableIssueCommentsClient.cs 97.26% <100%> (+0.65%) ⬆️

@hnrkndrssn hnrkndrssn changed the title Add ability to specify 'since' request parameter to issue comments [WIP] Add ability to specify 'since' request parameter to issue comments Sep 21, 2019
@hnrkndrssn hnrkndrssn changed the title [WIP] Add ability to specify 'since' request parameter to issue comments Add ability to specify 'since' request parameter to issue comments Sep 21, 2019
Copy link
Member

@shiftkey shiftkey left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this @hnrkndrssn!

@shiftkey shiftkey merged commit 9e5a768 into octokit:master Sep 21, 2019
@shiftkey
Copy link
Member

release_notes: added since parameter to Issue.Comments.GetAllForIssue()

@nickfloyd nickfloyd added Type: Bug Something isn't working as documented and removed category: bug labels Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Issue Comments Since Request Parameter
3 participants