Skip to content

docs: add contributor onboarding guide#144

Merged
ved015 merged 4 commits intoXortexAI:mainfrom
Ankit-Kotnala:docs/add-contributing-guide
May 5, 2026
Merged

docs: add contributor onboarding guide#144
ved015 merged 4 commits intoXortexAI:mainfrom
Ankit-Kotnala:docs/add-contributing-guide

Conversation

@Ankit-Kotnala
Copy link
Copy Markdown
Contributor

@ishaanxgupta please review this.

Summary

  • Added a root-level CONTRIBUTING.md as the centralized onboarding guide for new contributors.
  • Documented development setup, environment configuration, local server run command, testing, linting, coding standards, documentation standards, and PR expectations.

Notes

  • Documentation-only change.
  • No runtime code modified.

Comment thread CONTRIBUTING.md Outdated
Install the project with development dependencies:

```bash
pip install -e ".[dev]"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

pip install -e . is enough I think

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Ankit-Kotnala pls resolve this

Comment thread CONTRIBUTING.md Outdated

If you need the full local stack, use the Docker Compose files in `docker/`.

## Testing
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lets leave this for now as we dont have a proper test suite

Comment thread CONTRIBUTING.md Outdated
- ...
```

## Review Process
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can remove this section for now

@ishaanxgupta
Copy link
Copy Markdown
Member

Rest Looks good to me

@ved015
Copy link
Copy Markdown
Contributor

ved015 commented May 5, 2026

@Ankit-Kotnala pls make these changes so that we can add this to the merge queue

Removed sections on creating environment files, running locally, testing, and review process from CONTRIBUTING.md.
@Ankit-Kotnala Ankit-Kotnala requested a review from ved015 as a code owner May 5, 2026 15:07
Copy link
Copy Markdown
Contributor

@ved015 ved015 left a comment

Choose a reason for hiding this comment

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

pls resolve the suggestion which @ishaanxgupta raised then we can merge this😁

@ved015 ved015 merged commit 8b25da9 into XortexAI:main May 5, 2026
1 check passed
@ved015
Copy link
Copy Markdown
Contributor

ved015 commented May 5, 2026

Thank you @Ankit-Kotnala for the contribution

@Ankit-Kotnala
Copy link
Copy Markdown
Contributor Author

Thank you @ved015 and @ishaanxgupta for being so spontaneous. Would catch you guys in another issue prolly.

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