Skip to content

chore: initialise develop branch and branch management infrastructure#4

Open
AUREX-ML wants to merge 23 commits into
mainfrom
develop
Open

chore: initialise develop branch and branch management infrastructure#4
AUREX-ML wants to merge 23 commits into
mainfrom
develop

Conversation

@AUREX-ML
Copy link
Copy Markdown
Owner

Summary

This PR establishes the develop integration branch and implements the
full branch management infrastructure as documented in BRANCHING_STRATEGY.md.

Changes

  • ✅ Created develop branch from main
  • ✅ Added .github/dependabot.yml (targets develop, not main)
  • ✅ Added scripts/setup_branch_protection.sh
  • ✅ Added .github/CODEOWNERS
  • ✅ Branch protections applied to main and develop
  • ✅ Auto-delete merged branches enabled
  • ✅ Repository topics set

Checklist

  • Code follows project style guidelines
  • Documentation has been updated
  • All CI checks pass
  • Reviewed by at least one team member

Related

Resolves branch management gaps identified in repo audit (2026-05-14).

Copilot AI review requested due to automatic review settings May 14, 2026 13:26
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces repository governance infrastructure for a Git Flow–style workflow by adding a develop integration branch and wiring up automation/config to support protected-branch development and dependency update flows.

Changes:

  • Adds a GitHub CLI script to apply branch protection settings for main and develop.
  • Configures Dependabot to open update PRs against develop for Python and GitHub Actions.
  • Introduces CODEOWNERS to define ownership/review responsibility for key paths.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
scripts/setup_branch_protection.sh Adds an automated branch-protection setup script via gh api for main/develop.
.github/dependabot.yml Configures Dependabot to target the develop branch for updates.
.github/CODEOWNERS Defines code ownership rules intended to drive review routing/requirements.
Comments suppressed due to low confidence (1)

scripts/setup_branch_protection.sh:54

  • Same issue for develop: the required status check context "lint-and-test" doesn’t correspond to any existing check in .github/workflows/ci.yml (jobs are "Lint" and "Test"). This will make the protection rule ineffective or block merges unexpectedly; use the real check names (e.g., "CI / Lint" and "CI / Test").
echo '{
  "required_status_checks": {
    "strict": true,
    "contexts": ["lint-and-test"]
  },

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +19 to +23
echo '{
"required_status_checks": {
"strict": true,
"contexts": ["lint-and-test", "security-scan"]
},
Comment on lines +25 to +29
"required_pull_request_reviews": {
"required_approving_review_count": 1,
"dismiss_stale_reviews": true,
"require_code_owner_reviews": false
},
Comment thread scripts/setup_branch_protection.sh Outdated
Comment thread .github/CODEOWNERS
Comment on lines +5 to +19
# Global default — AUREX-ML reviews everything
* @AUREX-ML

# Core source code
src/ @AUREX-ML

# CI/CD and infrastructure
.github/ @AUREX-ML
docker-compose.yml @AUREX-ML
requirements.txt @AUREX-ML
pyproject.toml @AUREX-ML

# Documentation
docs/ @AUREX-ML
*.md @AUREX-ML
Comment thread .github/CODEOWNERS Outdated
Comment thread .github/CODEOWNERS Outdated
AUREX-ML and others added 3 commits May 14, 2026 16:32
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
@AUREX-ML AUREX-ML enabled auto-merge May 14, 2026 13:35
Comment thread tests/unit/test_forecaster.py Fixed
- fastapi 0.111.0 → 0.115.12 (pulls starlette≥0.47.2, fixes CVE-2024-47874 & CVE-2025-54121)
- pytest 8.2.0 → 9.0.3 (fixes CVE-2025-71176)
- black 24.4.2 → 26.3.1 (fixes CVE-2026-32274)
@AUREX-ML AUREX-ML mentioned this pull request May 14, 2026
Closed
Comment thread src/services/telemetry.py Fixed
Comment thread src/vpp/edge/mqtt_bridge.py Fixed
Comment thread tests/vpp/test_mqtt_bridge.py Fixed
aurex-ml and others added 5 commits May 15, 2026 03:30
…ore than once'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Ayiemba Rodgers  <aurexcv@gmail.com>
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