Glenn Lewis

Results 435 comments of Glenn Lewis

This also relates to: * #2618 * #3504

My understanding is that you wish to split `WebHookPayload` and `PushEvent` into two separate structs so that fields that are no longer populated will no longer exist in the struct,...

> Then maybe it would be enough to update the comment in the code that the deprecation is (currently) relevant just for events API. That sounds like an excellent idea....

@stevehipwell and/or @alexandear - do you approve this PR now?

> Does the issue need to be assigned or can I push a PR for this? I was wondering how i would be separating the PRs because changes would be...

> Hi, due to work constraints, I am not too sure if I can finish this issue by myself, or at least anytime soon so if anyone wants to continue...

> Hi, I’d like to work on this issue. Could you please assign it to me? Thank you, @sarthakw7! It is yours.

> I want to help with this issue [@gmlewis](https://github.com/gmlewis) Thank you for your interest, @alkak95! @sarthakw7 - are you OK if I switch ownership of this issue to @alkak95?

I apologize that this wasn't clear before, but in this issue we are only talking about parameters of _exported_ methods, not internal methods. There is nothing inherently wrong or bad...