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 function to delete pending reviews on a pull request #1897

Merged

Conversation

johnsc1
Copy link
Contributor

@johnsc1 johnsc1 commented Mar 26, 2021

This pull request will add a delete review function to resolve Issue #1856 . The new function delete is in github/PullRequestReview.py. It uses the Github API call linked in the issue to delete a review on a pull request. The test for this function is passing.

@codecov-io
Copy link

Codecov Report

Merging #1897 (7ef516d) into master (aa24030) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1897      +/-   ##
==========================================
- Coverage   98.87%   98.86%   -0.02%     
==========================================
  Files         108      108              
  Lines       11044    11062      +18     
==========================================
+ Hits        10920    10936      +16     
- Misses        124      126       +2     
Impacted Files Coverage Δ
.../runner/work/PyGithub/PyGithub/github/Requester.py 97.83% <0.00%> (-0.64%) ⬇️
.../runner/work/PyGithub/PyGithub/github/MainClass.py 95.22% <0.00%> (-0.42%) ⬇️
...unner/work/PyGithub/PyGithub/github/Permissions.py 100.00% <0.00%> (ø)
...r/work/PyGithub/PyGithub/github/GithubException.py 100.00% <0.00%> (ø)
...work/PyGithub/PyGithub/github/PullRequestReview.py 100.00% <0.00%> (ø)
...work/PyGithub/PyGithub/github/AuthenticatedUser.py 99.00% <0.00%> (+<0.01%) ⬆️
...runner/work/PyGithub/PyGithub/github/Repository.py 97.41% <0.00%> (+<0.01%) ⬆️
.../runner/work/PyGithub/PyGithub/github/PublicKey.py 87.09% <0.00%> (+4.33%) ⬆️

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 aa24030...7ef516d. Read the comment docs.

@johnsc1 johnsc1 marked this pull request as ready for review April 12, 2021 18:29
@s-t-e-v-e-n-k s-t-e-v-e-n-k merged commit c8a945b into PyGithub:master Oct 24, 2021
simkimsia added a commit to simkimsia/PyGithub that referenced this pull request Oct 24, 2021
commit 6452ddf
Author: Steve Kowalik <steven@wedontsleep.org>
Date:   Sun Oct 24 15:21:31 2021 +1100

    Add Repository.rename_branch method (PyGithub#2089)

    The GitHub API exposes an endpoint to rename a branch, so we should
    support calling it. Sadly, there is not enough information to add
    that method to the Branch class, so expose it in the Repository object.

    Fixes PyGithub#1901

commit c8a945b
Author: Claire Johns <42869556+johnsc1@users.noreply.github.com>
Date:   Sun Oct 24 00:15:31 2021 -0400

    Add function to delete pending reviews on a pull request (PyGithub#1897)

    Add a delete method to PullRequestReview to allow dismissing them.

    Fixes PyGithub#1856

    Co-authored-by: bagashvilit <bagashvilit@allegheny.edu>
    Co-authored-by: WonjoonC <chos@allegheny.edu>

commit f1faf94
Author: Steve Kowalik <steven@wedontsleep.org>
Date:   Fri Oct 22 08:39:31 2021 +1100

    Cover all code paths in search_commits (PyGithub#2087)

    The search_commits method was only very lightly tested, meaning over
    half of it was not covered. Write another test case, covering all code
    paths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Is there a way to delete a PR review?
5 participants