Skip to content

67 update reportmd with arvids coverage improvements#3279

Closed
HerodrawzZ wants to merge 25 commits into
python-discord:mainfrom
dd2480-spring-2025-group-1:67-update-reportmd-with-arvids-coverage-improvements
Closed

67 update reportmd with arvids coverage improvements#3279
HerodrawzZ wants to merge 25 commits into
python-discord:mainfrom
dd2480-spring-2025-group-1:67-update-reportmd-with-arvids-coverage-improvements

Conversation

@HerodrawzZ
Copy link
Copy Markdown

No description provided.

Strengthless and others added 25 commits February 15, 2025 03:48
…ith the latest consensus on manual counting
…f `on_command_error` function (#24)

Closes issue #23 by adding complexity analysis for on_command_error function

* docs(#23): add complexity analysis for on_command_error

Adds complexity analysis for on_command_error function loacted in ./bot/exts/backend/error_handler.py to the report

* docs(#23): Adds CCN of on_command_error function to report

I forgot to add the CCN in the last commit
…cello's refactoring (#26)

* docs(#25): Update `refactoring` section in `report.md` to include
refactoring in `invite.py`

Adds info on refactoring of the `apply_for` function in `invite.py`

* docs(#25): Add note on function change to report

Adds note on function change from `on_command_error` to `apply_for` for
the second part of the assignment

* docs(#25): fix typo in function name in report

* docs(#25): implement changes from review

Implemnts changes requested by @Strengthless
* docs(#39): added all questions for infraction_edit

* docs: update README.md with consistency changes

---------

Co-authored-by: Johan Nilsson <ni5@ug.kth.se>
Co-authored-by: Kim Kam <dickykam3@gmail.com>
* docs(#35): add invite.py coverage to report

Add coverage invite.py info to report

* docs(#35): add missing lines to report

Adds missing lines for coverage improvement for invite.py to report

* docs(#35): add reamining info need for report

Adds test description and link to PR to report for coverage improvement
of `apply_for`
Co-authored-by: Johan Nilsson <ni5@ug.kth.se>
Co-authored-by: Johan Nilsson <ni5@ug.kth.se>
@HerodrawzZ HerodrawzZ closed this Feb 21, 2025
@HerodrawzZ
Copy link
Copy Markdown
Author

Sry <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants