Skip to content
This repository was archived by the owner on Dec 26, 2022. It is now read-only.

Conversation

@zunda-arrow
Copy link
Member

Changes

  • adds: Better logging. Please tell me if you are confused by the messages so I can make them better.
  • fixed: Commands are no longer removed every time the bot is loaded
  • improvements: Commands with the same name and different type and same name now work properly.

Check off the following

  • I have tested my changes with the current requirements
  • My Code follows the pep8 code style.

@codecov
Copy link

codecov bot commented Dec 18, 2021

Codecov Report

Merging #317 (cabd82e) into main (7343160) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #317   +/-   ##
=======================================
  Coverage   90.00%   90.00%           
=======================================
  Files           9        9           
  Lines         100      100           
=======================================
  Hits           90       90           
  Misses         10       10           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7343160...cabd82e. Read the comment docs.

Copy link
Member

@Enderchief Enderchief left a comment

Choose a reason for hiding this comment

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

can you also format these files with black if not formatted

@zunda-arrow zunda-arrow merged commit e1841a1 into Pincer-org:main Dec 21, 2021
@zunda-arrow zunda-arrow deleted the command_register_fix branch December 21, 2021 20:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants