Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @Vamsi-o
18 changes: 18 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
## Summary

<!-- Short description of the change. -->

## What changed
- Describe what you changed and why.

## How to test
- Steps to reproduce / test this PR locally.

## Checklist
- [ ] I opened this PR from a feature branch (not main)
- [ ] CI builds and tests pass (no CI configured yet)
- [ ] I added/updated tests if applicable
- [ ] I added documentation if applicable

## Reviewers
- @Vamsi-o (code owner) will be automatically requested to review.
22 changes: 22 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# Contributing to BuildFlow

Thank you for contributing! To keep the project stable and reviewable, follow these rules:

1. Branches and commits
- Always create a branch for your work; do not commit directly to `main`.
- Use descriptive branch names like `feature/add-x` or `fix/issue-123`.

2. Pull Requests
- Open a Pull Request from your branch into `main`.
- Use the PR template (.github/PULL_REQUEST_TEMPLATE.md). Describe the change and how to test it.
- A Code Owner review is required (see .github/CODEOWNERS) — @Vamsi-o is requested by default.
- The `ci` status check is not required yet; CI will be added later.

3. Reviews and merging
- A code owner (or an assigned reviewer) should approve the PR before merging.
- Once approved, a maintainer can merge the PR into `main`.

4. Issues
- Open an issue before implementing non-trivial changes to discuss design and scope.

If you'd like stricter automation (e.g., required status checks, automatic merges after approval), we can add CI and automation in a follow-up PR.