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

Fixed multiple module descriptions #355

Merged
merged 9 commits into from
Aug 5, 2022

Conversation

CozyMeli-gitv2
Copy link
Contributor

Describe the pull
I fixed typos under multiple module descriptions to fit the style lambda maintains with its module descriptions and made them more clear and explanatory.

Describe how this pull is helpful
This pull improves multiple module descriptions, making them easier to understand and more explanatory. It helps alleviate potential confusion if someone is unfamiliar with the function of a module, and keeps the same overall description style.

Additional context
None to be had

Made descriptions clearer and more concise.
Changed description to describe the function of the module and removed punctuation to fit the overall description style.
Changed a repeat use of the word in to within, to stay grammatically correct, and removed punctuation to stay within the style of other module's descriptions.
The description seemed to be mistakenly copied over from logout logger, which was a cause of confusion. Removed the incorrect description and wrote a new correct one.
Changed might to attempt, as might is more informal and does not fit the use case as well.
Edited description as the module only allows you to add delay, not cancel.
Changed to be more formal and shorter
Changed description to describe the function of the module and be more explanatory.
Improved description and made it more descriptive.
@Avanatiker
Copy link
Member

thank you a lot :)
hehe "Logs when a player leaves the game" xD fail

@Avanatiker Avanatiker merged commit 7b223c8 into lambda-client:master Aug 5, 2022
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.

2 participants