Change type of HookDelivery.InstallationID to Int64.

[pierce-m]

Resolves #2234

[codecov]

Codecov Report

Merging #2235 (8ba2c80) into master (1d9884e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2235   +/-   ##
=======================================
  Coverage   97.79%   97.79%           
=======================================
  Files         113      113           
  Lines       10156    10156           
=======================================
  Hits         9932     9932           
  Misses        156      156           
  Partials       68       68           

Impacted Files	Coverage Δ
github/repos_hooks_deliveries.go	90.24% <ø> (ø)

---

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 1d9884e...8ba2c80. Read the comment docs.

[gmlewis]

I'm wondering why these fields are pointers but don't have omitempty?

[gmlewis]

I've looked through the code and can't find an explanation for why the fields in the HookDelivery do not have an omitempty even though they are all references.

So while we are breaking the API, please go ahead and add omitempty to all the fields in lines 20-30. Thank you!

[pierce-m]

@gmlewis yeah it looks like it was added on the Request and Response fields 🤔, maybe because this struct isn't used as input (as far as I can tell)? Anyways, added! 👍

[gmlewis]

Thank you, @pierce-m !
LGTM.

Awaiting second LGTM from any other repo contributor before merging.

[pierce-m]

Is there anyone else who can provide review? Doesn't seem like @wesleimp is very active any more.

[Parker77]

LGTM.

[gmlewis]

Thank you, @Parker77 !
Merging.