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

Fix: Some plugins had wrong run signature #161

Merged
merged 3 commits into from
Oct 8, 2022
Merged

Fix: Some plugins had wrong run signature #161

merged 3 commits into from
Oct 8, 2022

Conversation

ogabrielluiz
Copy link
Contributor

Issue #153

Description

This PR fixes issue #153 by updating the run functions on Plugins joins, log, and uptime.

How Has This Been Tested?

After the fixes, I reran the project and the errors were not happening anymore.

Additional context

Add any other context or screenshots about the pull request here.

Checklist

  • I have read, understood and followed our Contributing Guide
  • My code follows the code style of this project
  • I have updated the documentation accordingly
  • I added my country flag to the README.md

Code of Conduct

By submitting this issue, you agree to follow our Code of Conduct

  • I agree to follow this project's Code of Conduct

@MrTanoshii MrTanoshii self-requested a review October 8, 2022 22:09
@MrTanoshii MrTanoshii added bug Something isn't working plugin Related to honeybot plugins labels Oct 8, 2022
@MrTanoshii MrTanoshii added this to the Repository housekeeping milestone Oct 8, 2022
@MrTanoshii
Copy link
Member

Closes #153

@MrTanoshii MrTanoshii merged commit 2ba7afe into pyhoneybot:master Oct 8, 2022
@MrTanoshii
Copy link
Member

Thank you!

MrTanoshii pushed a commit that referenced this pull request Oct 23, 2022
* Fix: Some plugins had wrong run signature

Issue #153
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest-approved plugin Related to honeybot plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants