added /kick and /ban commands by doomed-neko · Pull Request #92 · allthingslinux/tux

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

kzndotsh merged 3 commits intoallthingslinux:mainfrom

doomed-neko:moderation

Mar 30, 2024

Merged

added /kick and /ban commands#92

kzndotsh merged 3 commits intoallthingslinux:mainfrom

doomed-neko:moderation

Conversation

@doomed-neko

Copy link Copy Markdown

Contributor

fixed the ping command by removing unneeded role check

@doomed-neko doomed-neko changed the title added /kick command added /kick and /ban commands

Mar 30, 2024

@electron271

Copy link Copy Markdown

Member

honestly not really sure why sonarcloud is complaining

@doomed-neko

Copy link Copy Markdown

Contributor Author

honestly not really sure why sonarcloud is complaining

it's about some duplicate code, but I don't see the problem since both files have the same functionality with small changes to method names

@sonarqubecloud

Copy link Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
39.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@kzndotsh kzndotsh merged commit 37f49d0 into allthingslinux:main

Mar 30, 2024

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

Reviewers

@kzndotsh kzndotsh Awaiting requested review from kzndotsh

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@doomed-neko @electron271 @kzndotsh