feat: allow the word invalid for clearer error descriptions - Githubissues

EddieHubCommunity / EddieBot

Inclusive words Discord bot (no more "hey guys")

http://discord.eddiehub.org

MIT License

173 stars 138 forks source link

feat: allow the word invalid for clearer error descriptions #751

Closed christophermorin closed 1 year ago

christophermorin commented 2 years ago

Fixes Issue

Closes #748

Changes proposed

Allows the word 'invalid' to be used.

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Please ignore if changing the banned words list is frowned upon. This seemed like an easier issue for a beginner so I had a crack at it.

loftwah commented 1 year ago

This looks good to me, although it isn't apparent why there are two lists of words. What is the difference between line 8 and line 13?

For example, fellow is found in both of them.