-
-
Notifications
You must be signed in to change notification settings - Fork 406
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
pronouns: add .clearpronouns
command
#2154
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 |
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 qa
reports a bunch of issues but unrelated to my change.Checklist
make qa
(runsmake quality
andmake test
)