Simplify the bot contributing guide#939
Merged
Merged
Conversation
✅ Deploy Preview for pydis-static ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
ChrisLovering
force-pushed
the
Simplify-the-bot-contributing-guide
branch
5 times, most recently
from
3fa33da to
adb81c4
Compare
Contributor
|
Feel free to assign me when this is ready for review. |
ChrisLovering
force-pushed
the
Simplify-the-bot-contributing-guide
branch
from
adb81c4 to
7064ab7
Compare
ChrisLovering
force-pushed
the
Simplify-the-bot-contributing-guide
branch
from
7064ab7 to
5c0e04f
Compare
jchristgit
approved these changes
Apr 17, 2023
shtlrs
requested changes
Apr 22, 2023
Contributor
shtlrs
left a comment
shtlrs
left a comment
There was a problem hiding this comment.
Looking great, 2 small comments.
|
|
||
| The site, however, is a mandatory service for the bot. | ||
| Refer to the [previous section](#with-the-bot-running-locally) and the [site contributing guide](../site) to learn how to start it on the host, in which case you will need to change `urls.site` in `.env.server` to wherever the site is being hosted. | ||
| If you want to read about how to make debugging with an IDE a easier, or for additional running methods, check out our [extended configuration guide](../bot-extended-configuration-options). |
Contributor
There was a problem hiding this comment.
I think this should be a note, so that it jumps to the eye easily
Member
Author
There was a problem hiding this comment.
|
|
||
| Once the previous steps are done, all that is left is to: | ||
| 1. Create a new channel | ||
| 2. Choose the `Forum` type |
Contributor
There was a problem hiding this comment.
This currently renders like this
Member
Author
There was a problem hiding this comment.
ChrisLovering
force-pushed
the
Simplify-the-bot-contributing-guide
branch
from
5c0e04f to
a5b7c50
Compare
Also update the guide to be far simpler, and less words to read for our contributors to get setup.
ChrisLovering
force-pushed
the
Simplify-the-bot-contributing-guide
branch
from
a5b7c50 to
facb030
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This splits out the manual configuration setup from the main guide, along with updating the guide to be far simpler, and less words to read for our contributors to get setup.