Skip to content

Conversation

brianferri
Copy link
Contributor

@brianferri brianferri commented Jun 9, 2022

Please describe the changes this PR makes and why it should be merged:
Step one of the shortening: A project for making the existing code shorter and cleaner, while integrating pre existing utilities which never got to be used (util/getChannel.js) ;-;
Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes

NOTES:
The changes presented here bring a lot to the table, apart from the usage of the pre existing .util/getChannel.js also numerous little mods have been made to make the code cleaner and more effective.
This is not valid for the indentation as I was told to do so against my will.
Have fun finding all the bugs that may or may not exists within this PR.
The commands have been tested thoroughly enough to see if they actually work, but given this is a big commit there still may be some things we forgot. If you do find anything be sure to let us know through the discord support server or a new issue!

Step one of the `shortening`: A project for making the existing code shorter and cleaner, while integrating pre existing utilities which never got to be used (util/getChannel.js) ;-;
@brianferri brianferri marked this pull request as draft June 9, 2022 12:18
@brianferri
Copy link
Contributor Author

there will be more changes being added to this in order to shortify each command, so marking this as a draft

@JotaroKujo0525
Copy link
Collaborator

this repo is being flooded by prs ngl, anw i praise your english spellings here, you did a nice job

@brianferri
Copy link
Contributor Author

this repo is being flooded by prs ngl, anw i praise your english spellings here, you did a nice job

It's a revolution Heh ⌓‿⌓

Copy link
Contributor Author

@brianferri brianferri left a comment

Choose a reason for hiding this comment

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

The changes presented here bring a lot to the table, apart from the usage of the pre existing .util/getChannel.js also numerous little mods have been made to make the code cleaner and more effective.
This is not valid for the indentation as I was told to do so against my will.
Have fun finding all the bugs that may or may not exists within this PR.
The commands have been tested thoroughly enough to see if they actually work, but given this is a big commit there still may be some things we forgot. If you do find anything be sure to let us know through the discord support server or a new issue!

@brianferri brianferri marked this pull request as ready for review June 9, 2022 17:28
@SudhanPlayz
Copy link
Owner

There are too many changes. The code is working fine?

@brianferri
Copy link
Contributor Author

There are too many changes. The code is working fine?

Yes, everything has been tested with help from fellow contributors while we were in a VC. As far as the functionalities of each commands goes everything is working as expected

@SudhanPlayz
Copy link
Owner

is it ready for merge or still needs commit?

@brianferri
Copy link
Contributor Author

is it ready for merge or still needs commit?

everything is ready

@SudhanPlayz SudhanPlayz merged commit 4218fa8 into SudhanPlayz:v5 Jun 10, 2022
@SudhanPlayz
Copy link
Owner

image
Thanks for the contribution!

@brianferri brianferri deleted the patch-1 branch June 10, 2022 12:51
@LewdHuTao LewdHuTao mentioned this pull request Mar 1, 2023
2 tasks
gitignore64x pushed a commit to gitignore64x/Discord-MusicBot that referenced this pull request Dec 20, 2024
* `Shortening`

Step one of the `shortening`: A project for making the existing code shorter and cleaner, while integrating pre existing utilities which never got to be used (util/getChannel.js) ;-;
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.

3 participants