fix: Installation Struct Missing for CommitCommentPayload, IssueCommentPayload and IssuesPayload

[Fish-Nullify]

This pull request fixes issue #177

[coveralls]

coverage: 88.639% (-1.1%) from 89.716% when pulling e8574e3 on Fish-Nullify:issues-and-commits-installation-id into 69430a8 on go-playground:master.

[robinlieb]

Hi @Fish-Nullify,
Since this field is optional on the mentioned payloads, would it make sense to treat the installation struct accordingly? Something like:

Installation *struct {
	ID int64 `json:"id"`
} `json:"installation,omitempty"`

@deankarn do you have an opinion on that?

[Fish-Nullify]

Hi @robinlieb,

Thanks for the suggestion and I have provided an update to the installation structs for the specified payloads. I can easily revert this if necessary.

[deankarn]

@robinlieb I agree, if it’s optional then let’s make it a pointer.

@Fish-Nullify omitempty only works for Serialization in Go and not Deserialization.

[robinlieb]

@Fish-Nullify could you them please make the installation struct a pointer and remove the omitempty? Like:

Installation *struct {
	ID int64 `json:"id"`
} `json:"installation"`

[Fish-Nullify]

@robinlieb sorry for the late reply was away, but made the installation struct a pointer and removed the omitempty