Skip to content

Add CONTRIBUTING.md#89

Open
arpitjain099 wants to merge 4 commits into
dbetchkal:mainfrom
arpitjain099:docs/contributing
Open

Add CONTRIBUTING.md#89
arpitjain099 wants to merge 4 commits into
dbetchkal:mainfrom
arpitjain099:docs/contributing

Conversation

@arpitjain099

@arpitjain099 arpitjain099 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

No contributor guidelines exist yet. Adds a basic CONTRIBUTING.md covering dev setup, running tests, and PR expectations.

Signed-off-by: arpitjain099 <arpitjain099@gmail.com>

@elliott-ruebush elliott-ruebush left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good idea! I'm going to look at updating this PR and potentially adding an AGENTS.md as well.

CONTRIBUTING defers setup to the README and style to AGENTS.md. AGENTS.md
documents repo layout, science-vs-plumbing conventions, and testing patterns.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread CONTRIBUTING.md

## Setup

Install as described in the [README](README.md#installation). Use `pip install -e ".[dev]"` to include test tools.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This pip install -e ".[dev]" is pending the merge of #87

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right, good call. I'll update this to reference the current setup steps until #87 lands, then we can swap it in.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

To be clear, I'll plan to merge #87 first, then merge #89

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