dataclass decorator

[adsharma]

No description provided.

[mivade]

Thanks for your contribution! This looks fairly good as is but I think we want the included decorator to take the same arguments as the upstream one.

[adsharma]

ping

[mivade]

Sorry, I haven't had time to complete reviewing this yet. At a glance I think this looks fine but I want to test it out before merging.

[mivade]

Sorry again for the delay. This looks good to me apart from some formatting issues. However since I don't have CI setup yet to automatically check those I'll merge and can update that later.

Thanks again for your contribution!

[adsharma]

I run black against all my code these days. But yeah - there could be gratuitous whitespace changes caused potentially by different people using different versions of black. Thanks for reviewing and merging!

…

On Sun, Mar 14, 2021 at 8:55 AM Mike DePalatis ***@***.***> wrote: Sorry again for the delay. This looks good to me apart from some formatting issues. However since I don't have CI setup yet to automatically check those I'll merge and can update that later. Thanks again for your contribution! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#9 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFA2A3P6JBNBULKFRLO75LTDTL5ZANCNFSM4YKWLGKA> .