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.

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

Added usage examples to !module command #1484

Merged
merged 5 commits into from
Oct 3, 2021
Merged

Added usage examples to !module command #1484

merged 5 commits into from
Oct 3, 2021

Conversation

zneix
Copy link
Contributor

@zneix zneix commented Sep 27, 2021 •
edited

Pull request checklist:

  • CHANGELOG.md was updated, if applicable
  • Documentation in docs/ or install-docs/ was updated, if applicable
  • I have tested all changes

Closes #1483

RAnders00
RAnders00 requested changes Sep 27, 2021
View reviewed changes
Copy link
Contributor

@RAnders00 RAnders00 left a comment

Choose a reason for hiding this comment

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

Would like to see a changelog entry, but looks good otherwise! Thank you!

@zneix
Copy link
Contributor Author

zneix commented Sep 28, 2021 •
edited

Would like to see a changelog entry, but looks good otherwise! Thank you!

I will of course add one! That's why the PR is still marked as a WIP draft as I wanted to commit and push before I went to bed yesterday.

@zneix zneix marked this pull request as ready for review September 30, 2021 22:07
@RAnders00 RAnders00 enabled auto-merge (squash) October 3, 2021 07:39
@RAnders00 RAnders00 merged commit 50ca20e into pajbot:master Oct 3, 2021
@zneix zneix deleted the zneix/chore/add-module-command-examples branch December 22, 2021 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RAnders00 RAnders00 RAnders00 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add detailed examples for !module command
2 participants
@zneix @RAnders00