chore!: Add sliceofpointers custom linter

[gmlewis]

BREAKING CHANGE: ListOAuthApps now returns ([]*OAuthApp, error) instead of ([]OAuthApp, error).

This PR should help developers use a consistent style by not writing []*string or []T in their PRs.
It should also ease the maintenance burden by not having to repeat this over and over again.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.03%. Comparing base (27bd175) to head (15174df).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3447   +/-   ##
=======================================
  Coverage   91.03%   91.03%           
=======================================
  Files         179      179           
  Lines       15532    15532           
=======================================
  Hits        14139    14139           
  Misses       1221     1221           
  Partials      172      172           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gmlewis]

@stevehipwell - do you have time for a code review? Thank you!

[ldez]

I'm curious: what is the reasoning behind using slice of pointer of struct inside of slice of struct?

[gmlewis]

I'm curious: what is the reasoning behind using slice of pointer of struct inside of slice of struct?

• replace []Issue with []*Issue #180
• Why are all the responses using pointers #1429
• Policy on using pointers/values in structs for receiving data from GitHub servers. #537
• Using pointers all over the places? #79
• add Getter methods for data structures #45
• issue setting empty values #19

[gmlewis]

Thank you, @ldez!
Merging.