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 command execution for subcommands in camelCase #612

Merged
merged 2 commits into from
Mar 12, 2022

Conversation

dlsf
Copy link
Member

@dlsf dlsf commented Feb 13, 2022

Fixes #609

@dlsf dlsf added the bug Something isn't working label Feb 13, 2022
@dlsf dlsf requested a review from PeachesMLG February 13, 2022 22:46
@dlsf dlsf self-assigned this Feb 13, 2022
@dlsf
Copy link
Member Author

dlsf commented Feb 15, 2022

I know that a stream inside a stream isn't particularly beautiful, but the alternatives are pretty dumb

@PeachesMLG
Copy link
Member

Yeah this does look kinda dum

@PeachesMLG
Copy link
Member

Instead of map collect contains, would Stream#anyMatch look better?

@PeachesMLG PeachesMLG merged commit 5cca7c7 into master Mar 12, 2022
@PeachesMLG PeachesMLG deleted the fix/camel-case-subcommand-execution branch March 12, 2022 20:29
SoKnight pushed a commit to STAR-Minecraft/IridiumSkyblock that referenced this pull request Mar 31, 2022
…nt#612)

* Fix command execution for subcommands in camelCase

* Shorten code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'/is home currentoffset' Isn't doing anything.
2 participants