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

Update Build-A-Bot section for simplified setup #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

harrrson
Copy link

@harrrson harrrson commented Mar 5, 2022

No description provided.

@UndarkAido
Copy link
Member

I like everything about this except for that last comma, maybe the last part of that sentence should be rewritten instead?

@harrrson
Copy link
Author

harrrson commented Mar 5, 2022

As for comma, for me it looks grammatically correct. Tomorrow i'll look how maybe rewrite it, and maybe i'll check whole docs for improvements :)

@UndarkAido
Copy link
Member

UndarkAido commented Mar 5, 2022

I'm pretty certain for example:

It makes use of select to provide a handy menu system for picking plugins, and configures the bot to use them.

If you remove "and branches" it's definitely misplaced, and I don't think the pair is supposed to change the grammatical structure of the sentence.

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