Turns out I do care about formatting a little. We have a lot of builders and chained methods.
I think it is easier to follow long chains when lined up veritcally.
This may be controversial but it we're doing it. Having code formatting needs to be consistent
and a non-issue during code review. I'm willing modify the configuration if people see a strong
need, but formatting needs to be present and enforced.
- Created separate class for GHIssueEvent instead of nested classes.
- Renamed test class accordingly.
Other fixes:
- Changed id type to long.
- Updated test cases.
- Added toString() method.
