Skip to content

Added clear_user_list function to wrapper#6

Open
kalinon wants to merge 3 commits intojbreeden:masterfrom
spoved:master
Open

Added clear_user_list function to wrapper#6
kalinon wants to merge 3 commits intojbreeden:masterfrom
spoved:master

Conversation

@kalinon
Copy link

@kalinon kalinon commented Feb 4, 2018

Added clear_user_list function to wrapper

@jbreeden
Copy link
Owner

jbreeden commented Apr 3, 2018

For me, the rubocop changes are a bit much, even though it mostly looks like an improvement. I'd be happy to merge the additional clear_user_list wrapper on its own. I can squash the extra changes out if need be, but I'm guessing I'll procrastinate even longer if that's the case :P

I just need to find some time for testing, and maybe setting up automated testing for this repo.

Thanks for the PR!

@kalinon
Copy link
Author

kalinon commented Apr 3, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants