Closed kagawaa closed 5 years ago
Everything looks good
Sorry I didn't merge earlier @kagawaa , I didn't see the Review Request. I'm not versed enough into GitHub, but how do we make sure there's a limited number of Kentik employees who can merge? (and that they're systematically added for review)
@grrrrreg i'm sure there is a repository setting some where that restricts the folks that can merge. but i think I don't have access to that.
We don't have this on now. Can add, similar to how kentik/sql-schema is set up.
On Mon, Feb 25, 2019 at 10:41 AM Aaron Kagawa notifications@github.com wrote:
@grrrrreg https://github.com/grrrrreg i'm sure there is a repository setting some where that restricts the folks that can merge. but i think I don't have access to that.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kentik/config-snippets/pull/1#issuecomment-467129425, or mute the thread https://github.com/notifications/unsubscribe-auth/AATQRApU5C1-rC1AJO6LGpYdjz3k3NoPks5vRC5tgaJpZM4bDbhW .
configs for netflow and snmp