-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Highlights from Users Banned in Chat #99
Conversation
The tests are failing because it expects the constants.Commands to be listed among the contribution commands configuration in the package.json file. However, the only commands that should exist in the package.json are those commands that are registered with vscode and executed by the user via the UI. We need to refactor the constants.Commands, as well as the package.json, to separate commands between registered commands (those commands registered with vscode, such as startChat) and internal commands (those commands that aren't registered with vscode, such as removeBannedHighlights). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my previous comment.
I should get to fixing this up tonight. I'll refactor and look to add tests for the internal commands too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…-surlydev docs: add surlydev as a contributor
Finished implementation that was started with mpj on https://twitch.tv/funfunfunction.
Big thanks to @parithon for helping simplify the code for removing highlights based on banned username with function overloads 👍
👉 https://gist.github.com/parithon/7b165c22e797186e7649378aa3a01351
Partially addresses #51