Skip to content
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

modules: add missing help messages #2035

Merged
merged 4 commits into from
Apr 11, 2021
Merged

modules: add missing help messages #2035

merged 4 commits into from
Apr 11, 2021

Conversation

sario528
Copy link
Contributor

@sario528 sario528 commented Mar 2, 2021

Description

What does this do? What problem does it solve? Does it resolve any GitHub issues?

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

sopel/modules/currency.py Outdated Show resolved Hide resolved
@dgw dgw added this to the 7.1.0 milestone Mar 2, 2021
@dgw dgw added the Tweak label Mar 2, 2021
@dgw dgw changed the title fix help message currency: add help message Mar 2, 2021
@dgw dgw changed the title currency: add help message modules: add missing help messages Mar 2, 2021
@sario528 sario528 marked this pull request as ready for review March 2, 2021 19:39
sario528 and others added 4 commits March 2, 2021 13:46
Co-authored-by: dgw <dgw@technobabbl.es>
Added help messages for .pronouns and .setpronouns
Add help message for .w/.wik/.wiki
Copy link
Member

@dgw dgw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made the history nice. Travis will finish its work between now and when this hits the top of our merge queue.

@dgw dgw merged commit b6304b5 into sopel-irc:master Apr 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants