8000 Add function for redelivering webhooks by DeviousLab · Pull Request #2061 · google/go-github · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add function for redelivering webhooks #2061

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

Merged
merged 5 commits into from
Sep 3, 2021
Merged

Add function for redelivering webhooks #2061

merged 5 commits into from
Sep 3, 2021

Conversation

DeviousLab
Copy link
Contributor
@DeviousLab DeviousLab commented Aug 24, 2021

Aims to resolve issue #2060. Let me know if I've done it wrong!

Signed-off-by: DeviousLab deviouslab@gmail.com

Fixes: #2060.

Signed-off-by: DeviousLab <deviouslab@gmail.com>
@google-cla google-cla 8000 bot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Aug 24, 2021
@codecov
Copy link
codecov bot commented Aug 24, 2021

Codecov Report

Merging #2061 (3e4a880) into master (a0448fc) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2061   +/-   ##
=======================================
  Coverage   97.75%   97.75%           
=======================================
  Files         107      107           
  Lines        9600     9612   +12     
=======================================
+ Hits         9384     9396   +12     
  Misses        150      150           
  Partials       66       66           
Impacted Files Coverage Δ
github/orgs_hooks_deliveries.go 100.00% <100.00%> (ø)

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 a0448fc...3e4a880. Read the comment docs.

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, @DeviousLab !
This is a good start. Please add a new unit test for the new method and make sure to run gofmt on your added code.

@gmlewis gmlewis changed the title Added function for redelivering webhooks #2060 Add function for redelivering webhooks Aug 24, 2021
DeviousLab and others added 4 commits August 25, 2021 06:55
As suggested by gmlewis

Co-authored-by: Glenn Lewis <6598971+gmlewis@users.noreply.github.com>
Signed-off-by: DeviousLab <deviouslab@gmail.com>
Signed-off-by: DeviousLab <deviouslab@gmail.com>
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, @DeviousLab !
LGTM.

Awaiting second LGTM before merging.

@gmlewis gmlewis requested a review from wesleimp August 25, 2021 14:16
Copy link
@Parker77 Parker77 left a comment

Choose a reason for hiding this comment

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

LGTM.

@gmlewis
Copy link
Collaborator
gmlewis commented Sep 3, 2021

Thank you, @Parker77 !
Merging.

@gmlewis gmlewis merged commit 159aed6 into google:master Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indication that the PR author has signed a Google Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New feature request: add Redeliver a delivery for an organization webhook
3 participants
0