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.

Sign up for GitHub

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

Jump to bottom

chungusify robocop #94

Open

leo60228 wants to merge 1 commit into reswitched:master from leo60228:chungusify

Contributor

leo60228 commented Apr 5, 2024

check_if_staff_or_ot has been broken for almost a year now. Oops.

Compliant with the "Works on My Machine" Certification Program:

Cause one code path in the code you're checking in to be executed. The preferred way to do this is with ad-hoc manual testing of the simplest possible case for the feature in question. Omit this step if the code change was less than five lines, or if, in the developer's professional opinion, the code change could not possibly result in an error.

The ideal solution would probably be to add an ot_channel channel ID to config.py, but this seems Good Enough.


          chungusify robocop

df33eae

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