-
-
Notifications
You must be signed in to change notification settings - Fork 409
pronouns: add .clearpronouns command
#2154
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
Conversation
.clearpronouns command.clearpronouns command
What issues would those be? It shouldn't report anything, and doesn't in CI. |
|
I get this: |
|
It appears you're using a version of Python older than what Sopel 8+ ( |
|
OK, all of that (especially |
|
Yes, that's it. Changing the default python to python3 now gives me a different error. Horray! |
|
So you need to |
dgw
left a comment
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.
Well and good once LGTM re-approves. Took care of the commit message style myself, same as I would have in #2140 if it hadn't been replaced.
Description
Adds the ability to clear one's pronouns.
Closes #2137
@dgw edit: It's because the checks were run on py2, which is no longer supported.make qareports a bunch of issues but unrelated to my change.Checklist
make qa(runsmake qualityandmake test)