Why-Bot icon indicating copy to clipboard operation
Why-Bot copied to clipboard

Why Bot v2

Open FusionSid opened this issue 3 years ago • 5 comments
trafficstars

Complete rewrite of the bot (for the like 1 millionth time) Rewrite starts at this commit: Link and not the one from 8 months ago.

FusionSid avatar Oct 30 '22 10:10 FusionSid

This is still a work in progress. I will merge once it has all the features that the main branch has.

FusionSid avatar Oct 30 '22 10:10 FusionSid

Plan for commit names

Im gonna try to somewhat follow the conventional commit method.

These are the types ill use plus i added some of my own:

  • fix: bug fix or command issue fix
  • feat: new command or bot feature
  • docs: comments, doc strings or actual docs being added
  • test: for pushing new tests
  • style: lighter refactor type changes like formatting and flake8 stuff
  • refactor: code refactoring
  • perf: performance improvement to command (ill probably use refactor or feat instead tho)
  • chore: anything else

Also i don't think i will include body and footers unless its a massive change. These changes have sorta begun like 10 commits ago but will be actual after this commit.

FusionSid avatar Nov 26 '22 07:11 FusionSid

TODO:

reduce complexity on these 3 methods

https://github.com/FusionSid/Why-Bot/blob/67235e81eeef8d768130dd88bad3c480eb618891/src/cogs/owner/dmreply.py#L54-L163

https://github.com/FusionSid/Why-Bot/blob/67235e81eeef8d768130dd88bad3c480eb618891/src/cogs/events/on_error.py#L15-L221

https://github.com/FusionSid/Why-Bot/blob/67235e81eeef8d768130dd88bad3c480eb618891/src/core/utils/formatters.py#L21-L117

FusionSid avatar Dec 07 '22 01:12 FusionSid

bruh i need to stop doing refactoring shit and actually code lmao

FusionSid avatar Dec 28 '22 11:12 FusionSid