Skip to content

Conversation

@seanmnguyen
Copy link

Closes #68

Overview

Adds the CONTRIBUTING.md file to the repo. Includes a Quick Start guide on how to set up the repo, install dependencies, and go through the rest of the setup. Includes Development Guidelines with a link to the STYLE_GUIDE.md

Testing

No testing

Screenshots / Screencasts

Preview of the markdown file (it's 3 pictures, sorry for the formatting):
contributing_1
contributing_2
contributing_3

Checklist

  • Code is neat, readable, and works
  • Code is commented where appropriate and well-documented
  • Commit messages follow our guidelines
  • Issue number is linked
  • Branch is linked
  • Reviewers are assigned (one of your tech leads)

Tip: You can make the issue and then check them after the fact or replace [ ] with [x] to check it!

Notes

Not sure if the STYLE_GUIDE.md is supposed to be incorporated into the CONTRIBUTING.md file or if just linking it is sufficient. Also, much of the content was taken from this repo's README.md and Cal Poly's H4I repo, so let me know if something doesn't make sense or conform to Operation's standards.

Adds the CONTRIBUTING.md file to the repo. Includes a Quick Start
guide on how to set up the repo, install dependencies, and go
through the rest of the setup. Includes Development Guidelines with
a link to the STYLE_GUIDE.md
Updated instructions for cloning the repo and branch naming conventions.

Signed-off-by: Sophia Chang <103406603+soramicha@users.noreply.github.com>
Copy link
Collaborator

@soramicha soramicha left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Sean!

@soramicha soramicha merged commit 4c35188 into develop Nov 24, 2025
1 check passed
@soramicha soramicha deleted the feature/68-add-contributing-markdown branch November 24, 2025 20:56
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