From acb819567de79cf1ab746514096dadab1f117626 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 13 Nov 2025 06:19:46 +0000 Subject: [PATCH 1/2] Initial plan From 7f2f3fd6214ebbe3b4ee791366558881e1be4d52 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 13 Nov 2025 06:23:24 +0000 Subject: [PATCH 2/2] chore: add CONTRIBUTING, PR template, and CODEOWNERS to require PRs Co-authored-by: Vamsi-o <216269458+Vamsi-o@users.noreply.github.com> --- .github/CODEOWNERS | 1 + .github/PULL_REQUEST_TEMPLATE.md | 18 ++++++++++++++++++ CONTRIBUTING.md | 22 ++++++++++++++++++++++ 3 files changed, 41 insertions(+) create mode 100644 .github/CODEOWNERS create mode 100644 .github/PULL_REQUEST_TEMPLATE.md create mode 100644 CONTRIBUTING.md diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 0000000..5f21e35 --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1 @@ +* @Vamsi-o diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 0000000..91139e1 --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,18 @@ +## Summary + + + +## 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. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md new file mode 100644 index 0000000..dd15ce9 --- /dev/null +++ b/CONTRIBUTING.md @@ -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.