Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix /p dispose #208

merged 1 commit into from Feb 2, 2019


None yet
3 participants
Copy link

cawloft commented Feb 2, 2019

  • This PR actually submits something - don't use this system as a messaging service!
  • This PR isn't duplicated - you can check if it is by scanning the small list - link is on the navigation bar for this repository.
  • This PR includes appropriate markdown for sections - e.g. code blocks for suggested code.
  • This PR description and comments in code is understandable - feel free to use your native language to write if you are not comfortable with English.
  • This PR has a single branch for itself in your fork - don't just commit to the master branch of your repository!
  • This PR has comments written in clear English - please don't write unreadable comments just for your eyes.

Pull Request description

It returns usage instead asking for confirmation when a player sends /p dispose

@Aericio Aericio added the PR:Bug Fixes label Feb 2, 2019


This comment has been minimized.

Copy link

Aericio commented Feb 2, 2019

looks good to me.

@jasonwynn10 jasonwynn10 merged commit fe4a9a3 into jasonwynn10:master Feb 2, 2019

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
poggit-ci/myplot Created PR build #68 (&55506): Lint passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.