Skip to content

Commit ff74370

Browse files
committed
• Updated best practices
• Updated index with new contribution guide link
1 parent 446d875 commit ff74370

File tree

4 files changed

+25
-19
lines changed

4 files changed

+25
-19
lines changed

docs/contributing/CONTRIBUTING.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@ To join, just raise your hand on the InvokeAI Discord server (#dev-chat) or the
1212
### Areas of contribution:
1313

1414
#### Development
15-
If you’d like to help with development, please see our [development guide](docs/contributing/.contribution_guides/development.md). If you’re unfamiliar with contributing to open source projects, there is a tutorial contained within the development guide.
15+
If you’d like to help with development, please see our [development guide](contribution_guides/development.md). If you’re unfamiliar with contributing to open source projects, there is a tutorial contained within the development guide.
1616

1717
#### Documentation
18-
If you’d like to help with documentation, please see our [documentation guide](docs/contributing/.contribution_guides/documenation.md).
18+
If you’d like to help with documentation, please see our [documentation guide](contribution_guides/documenation.md).
1919

2020
#### Translation
2121
If you'd like to help with translation, please see our [translation guide](docs/contributing/.contribution_guides/translation.md).
@@ -32,7 +32,7 @@ This project is a combined effort of dedicated people from across the world. [C
3232

3333
### Code of Conduct
3434

35-
The InvokeAI community is a welcoming place, and we want your help in maintaining that. Please review our Code of Conduct **<link to code of conduct>** to learn more.
35+
The InvokeAI community is a welcoming place, and we want your help in maintaining that. Please review our [Code of Conduct](../../CODE_OF_CONDUCT.md) to learn more.
3636

3737
### Support
3838

docs/contributing/contribution_guides/development.md

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ For more information, please review our area specific documentation:
99
* #### [InvokeAI Architecure](../ARCHITECTURE.md)
1010
* #### [Frontend Documentation](development_guides/contributingToFrontend.md)
1111
* #### [Node Documentation](../INVOCATIONS.md)
12+
* #### [Local Development](../LOCAL_DEVELOPMENT.md)
1213

1314
If you don't feel ready to make a code contribution yet, no problem! You can also help out in other ways, such as [documentation](documentation.md) or [translation](translation.md).
1415

@@ -20,44 +21,52 @@ There are two paths to making a development contribution:
2021

2122
*Regardless of what you choose, please post in the [#dev-chat](https://discord.com/channels/1020123559063990373/1049495067846524939) channel of the Discord before you start development in order to confirm that the issue or feature is aligned with the current direction of the project. We value our contributors time and effort and want to ensure that no one’s time is being misspent.*
2223

24+
## Best Practices:
25+
* Keep your pull requests small. Smaller pull requests are more likely to be accepted and merged
26+
* Comments! Commenting your code helps reviwers easily understand your contribution
27+
* Use Python and Typescript’s typing systems, and consider using an editor with [LSP](https://microsoft.github.io/language-server-protocol/) support to streamline development
28+
* Make all communications public. This ensure knowledge is shared with the whole community
29+
2330
## **How do I make a contribution?**
2431

2532
Never made an open source contribution before? Wondering how contributions work in our project? Here's a quick rundown!
2633

27-
1. Find a [good first issue](https://github.com/invoke-ai/InvokeAI/contribute) that you are interested in addressing or a feature that you would like to add. Then, reach out to our team in the [#dev-chat](https://discord.com/channels/1020123559063990373/1049495067846524939) channel of the Discord to ensure you are setup for success.
34+
Before starting these steps, ensure you have your local environment [configured for development](../LOCAL_DEVELOPMENT.md).
35+
36+
1. Find a [good first issue](https://github.com/invoke-ai/InvokeAI/contribute) that you are interested in addressing or a feature that you would like to add. Then, reach out to our team in the [#dev-chat](https://discord.com/channels/1020123559063990373/1049495067846524939) channel of the Discord to ensure you are setup for success.
2837
2. Fork the [InvokeAI](https://github.com/invoke-ai/InvokeAI) repository to your GitHub profile. This means that you will have a copy of the repository under **your-GitHub-username/InvokeAI**.
2938
3. Clone the repository to your local machine using:
3039

3140
```bash
32-
**git clone** https://github.com/your-GitHub-username/InvokeAI.git
41+
git clone https://github.com/your-GitHub-username/InvokeAI.git
3342
```
3443

3544
1. Create a new branch for your fix using:
3645

3746
```bash
38-
**git checkout -b branch-name-here**
47+
git checkout -b branch-name-here
3948
```
4049

4150
1. Make the appropriate changes for the issue you are trying to address or the feature that you want to add.
4251
2. Add the file contents of the changed files to the "snapshot" git uses to manage the state of the project, also known as the index:
4352

4453
```bash
45-
**git add insert-paths-of-changed-files-here**
54+
git add insert-paths-of-changed-files-here
4655
```
4756

4857
1. Store the contents of the index with a descriptive message.
4958

5059
```bash
51-
**git commit -m "Insert a short message of the changes made here"**
60+
git commit -m "Insert a short message of the changes made here"
5261
```
5362

5463
1. Push the changes to the remote repository using
5564

5665
```markdown
57-
**git push origin branch-name-here**
66+
git push origin branch-name-here
5867
```
5968

60-
1. Submit a pull request to the **main** branch of the InvokeAI repository.
69+
1. Submit a pull request to the **development** branch of the InvokeAI repository.
6170
2. Title the pull request with a short description of the changes made and the issue or bug number associated with your change. For example, you can title an issue like so "Added more log outputting to resolve #1234".
6271
3. In the description of the pull request, explain the changes that you made, any issues you think exist with the pull request you made, and any questions you have for the maintainer. It's OK if your pull request is not perfect (no pull request is), the reviewer will be able to help you fix any problems and improve it!
6372
4. Wait for the pull request to be reviewed by other collaborators.
@@ -76,5 +85,5 @@ For backend related work, please reach out to **@pyschedelicious, @blessedcoolan
7685

7786
## **What does the Code of Conduct mean for me?**
7887

79-
Our [Code of Conduct](../../CODE_OF_CONDUCT.md) means that you are responsible for treating everyone on the project with respect and courtesy regardless of their identity. If you are the victim of any inappropriate behavior or comments as described in our Code of Conduct, we are here for you and will do the best to ensure that the abuser is reprimanded appropriately, per our code.
88+
Our [Code of Conduct](CODE_OF_CONDUCT.md) means that you are responsible for treating everyone on the project with respect and courtesy regardless of their identity. If you are the victim of any inappropriate behavior or comments as described in our Code of Conduct, we are here for you and will do the best to ensure that the abuser is reprimanded appropriately, per our code.
8089

docs/index.md

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -221,14 +221,10 @@ get solutions for common installation problems and other issues.
221221

222222
Anyone who wishes to contribute to this project, whether documentation,
223223
features, bug fixes, code cleanup, testing, or code reviews, is very much
224-
encouraged to do so. If you are unfamiliar with how to contribute to GitHub
225-
projects, here is a
226-
[Getting Started Guide](https://opensource.com/article/19/7/create-pull-request-github).
227-
228-
A full set of contribution guidelines, along with templates, are in progress,
229-
but for now the most important thing is to **make your pull request against the
230-
"development" branch**, and not against "main". This will help keep public
231-
breakage to a minimum and will allow you to propose more radical changes.
224+
encouraged to do so.
225+
226+
[Please take a look at our Contribution documentation to learn more about contributing to InvokeAI.
227+
](contributing/CONTRIBUTING.md)
232228

233229
## :octicons-person-24: Contributors
234230

mkdocs.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,7 @@ nav:
143143
- Documentation: 'contributing/contribution_guides/documentation.md'
144144
- Translation: 'contributing/contribution_guides/translation.md'
145145
- Tutorials: 'contributing/contribution_guides/tutorials.md'
146+
- Local Development: contributing/LOCAL_DEVELOPMENT.md
146147
- Changelog: 'CHANGELOG.md'
147148
- Deprecated:
148149
- Command Line Interface: 'deprecated/CLI.md'

0 commit comments

Comments
 (0)