Add ban/unban user endpoints

nicklaw5 / helix

A Twitch Helix API client written in Go.

MIT License

243 stars 88 forks source link

Add ban/unban user endpoints #146

Closed diPhantxm closed 1 year ago

diPhantxm commented 1 year ago

add ban/unban endpoints
unit tests

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3268078973

16 of 18 (88.89%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 92.495%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
moderation.go	16	18	88.89%
<!--	Total:	16	18	88.89%	-->

Totals
Change from base Build 3260742499:	-0.05%
Covered Lines:	1294
Relevant Lines:	1399

Thanks for your comment. I've fixed data types as you said. Also added comment to unban user function and noticed one more defect in test function. Didn't notice your data type for time right away. It's actually my first time contributing to an open source project.

nicklaw5 commented 1 year ago

Thanks and welcome to open source development :slightly_smiling_face:

nicklaw5 / helix

Add ban/unban user endpoints #146

Pull Request Test Coverage Report for Build 3268078973

💛 - Coveralls