On 03-Jul-2023, at 4:54 PM, Franco Bugnano ***@***.***> wrote: First of all thank you for your contribution. Before being able to accept any PR, no matter how small, we require every contributor to sign either the Individual CLA or the corporate CLA that you find in this repo, and email it to ***@***.*** ***@***.***> Having said that, there are a couple of things that I don't like in this PR: Please don't reformat the sources, as it makes it nearly impossible to understand what are actual changes, and what are only automated formatting changes We don't use Firebase for push notifications on iOS, so removing the flutter_apns dependency results in breaking the push notifications on iOS — Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AICXBKJEYPMCTR3YHZYH5WDXOKXHBANCNFSM6AAAAAAZ4IU5AE>. You are receiving this because you authored the thread.

> On 03-Jul-2023, at 4:54 PM, Franco Bugnano ***@***.***> wrote: > > > First of all thank you for your contribution. > Before being able to accept any PR, no matter how small, we require every contributor to sign either the Individual CLA or the corporate CLA that you find in this repo, and email it to ***@***.*** ***@***.***> > Having said that, there are a couple of things that I don't like in this PR: > > Please don't reformat the sources, as it makes it nearly impossible to understand what are actual changes, and what are only automated formatting changes > We don't use Firebase for push notifications on iOS, so removing the flutter_apns dependency results in breaking the push notifications on iOS > — > Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AICXBKJEYPMCTR3YHZYH5WDXOKXHBANCNFSM6AAAAAAZ4IU5AE>. > You are receiving this because you authored the thread. >