feat(hook): add hook id to hook object

[ecrupper]

Couple related issues: go-vela/community#459 and go-vela/community#416

The hook_id is the field needed from GitHub to not only reach that deliveries page but also interact with any of their API endpoints related to hook deliveries. A hook_id is unique to any one hook within a repo, but since we already have an id field as well as a repo_id field, I chose address as the name. If anyone has a suggestion for another name, I am all ears!

I also cleaned up a couple of typos I noticed in the comments of build.go

[codecov]

Codecov Report

Merging #239 (b396bb0) into master (f85f4c1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head b396bb0 differs from pull request most recent head a4a1c96. Consider uploading reports for the commit a4a1c96 to get more accurate results

@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
+ Coverage   97.16%   97.17%   +0.01%     
==========================================
  Files          54       54              
  Lines        5828     5850      +22     
==========================================
+ Hits         5663     5685      +22     
  Misses        122      122              
  Partials       43       43              

Impacted Files	Coverage Δ
library/build.go	100.00% <ø> (ø)
database/hook.go	100.00% <100.00%> (ø)
library/hook.go	100.00% <100.00%> (ø)

[JordanSussman]

I chose address as the name. If anyone has a suggestion for another name, I am all ears!

I'm thinking hook_id would be less confusing compared to address.

[plyr4]

i agree with sussman on hook_id, consistent with the others repo_id etc

[plyr4]

although i see the scenario for hook.GetHookID and it not returning the ID of the hook to be confusing.

[plyr4]

some options, not all of them good

• hook_id
• delivery_id
• scm_id
• webhook_id
• payload_id
• address_id

[plyr4]

i kind of like delivery_id to be less ambiguous with "hook"

edit: i like webhook_id most, added "web" makes it easier to read than hook

[jbrockopp]

LGTM