JDA-Applications / JDA-Utilities

A series of tools and utilities for JDA to assist in bot creation
Apache License 2.0
218 stars 111 forks source link

Example Code Cleanup #83

Closed JonoAugustine closed 2 years ago

JonoAugustine commented 5 years ago

Pull Request

Pull Request Checklist

Please follow the following steps before opening this PR.
PRs that do not complete the checklist will be subject to denial for missing information.

Pull Request Information

Check and fill in the blanks for all that apply:

Description

I have improved the readability of the About and GuildList example commands so that newcomers can get a better grasp on how it all works.

Both

jagrosh commented 5 years ago

Right off the bat, please don't change the copyright, and please don't remove the original author from the file if you are modifying it.

JonoAugustine commented 5 years ago

woopsies, force of habit. ill change that

MinnDevelopment commented 5 years ago

It would be extremely helpful to include some kind of description or explanation as to what these changes exactly improve/change.

jagrosh commented 5 years ago

I also noticed you included a sun import, which is probably not very good practice considering some people might be building on OpenJDK or future releases. There is a JDA util called Checks that might have similar functionality and is already a dependency.

JonoAugustine commented 5 years ago

I also noticed you included a sun import, which is probably not very good practice considering some people might be building on OpenJDK or future releases. There is a JDA util called Checks that might have similar functionality and is already a dependency.

Thanks, was unaware of this

JonoAugustine commented 5 years ago

@MinnDevelopment updated description