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

Added Contributing.md file #250

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

ManoBharathi93
Copy link
Contributor

No description provided.

@dschonholtz
Copy link
Contributor

@Torantulino can we make this fairly high priority, but also add in contribution guidelines on what kind of code styling/basic standards we want?

```
git checkout -b "branch-name"
```
4. Make your changes to the code or documentation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
4. Make your changes to the code or documentation.
4. Make your changes to the code or documentation.
**Ensure each commit is small and addresses a SINGLE issue.** PRs with commits that sneak in extra tweaks will likely be rejected.

```
git add .
```

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Or, if using Git in VS-Code, select specific lines of code and commit them.

10. Select the branch you just pushed to and the branch you want to merge into on the original repository.

11. Add a description of your changes and click the "Create pull request" button.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Be as clear, descriptive and specific as possible.
- Use markdown formatting for a clear and well written post.

@Torantulino Torantulino merged commit 576322d into Significant-Gravitas:master Apr 6, 2023
codejach pushed a commit to codejach/auto-gpt that referenced this pull request Apr 7, 2023
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.

3 participants