Skip to content

chore: remove dev branch direct-push protection from git hooks#12

Open
SeanSan06 wants to merge 1 commit intodevfrom
seansan06/chore/remove-dev-push-protection
Open

chore: remove dev branch direct-push protection from git hooks#12
SeanSan06 wants to merge 1 commit intodevfrom
seansan06/chore/remove-dev-push-protection

Conversation

@SeanSan06
Copy link
Member

@SeanSan06 SeanSan06 commented Mar 13, 2026

Description

Removes direct-push protection for the exact dev branch in git hooks and updates hook messaging/docs to match the new branch policy. This change is needed so the dev environment is open to pushes while still keeping the logic that prevents direct pushes to protected branches. This also updates the necessary README.mds and related documentation.

Extra Note just for this PR: Created dev branch from main before opening this PR to align base branch strategy.

  • Related Issue number: (N/A)
  • Plaky feature name: (N/A)
  • Component/feature/system affected: Git hook workflow and developer onboarding/setup flow
  • Purpose of change: Refactor/workflow improvement

Changes

  • Removed the exact dev branch push-blocking rule from the pre-push hook
  • Kept protection for main, master, and team-dev pattern branches
  • Updated hook output messaging so the protected branch list no longer includes dev
  • Updated local hook setup script messaging to reflect the same policy

Be specific. Include:

  • Updated hook scripts and setup workflow scripts
  • Branch protection behavior alignment across local tooling
  • No dependency changes
  • No runtime business logic changes

Related

  • Issue: (N/A)
  • Plaky: (N/A)
  • Related PRs (optional): deepiri-platform PR + sister submodule PRs for this same hook policy update. There is a lot, so contact me for more information.

Testing

  • Verified local hook behavior no longer blocks direct pushes to dev
  • Verified local hook behavior still blocks protected branches (main/master/team-dev patterns)
  • Verified hook messaging output reflects the current protected branch policy
  • Verified branch used for PR is personal branch -> dev

Additional testing details:

  • Manual validation of hook output and commit diff per repo
  • No service/runtime behavior impact expected

Important Notes (Optional)

  • Known limitations: Hook behavior is local-client enforcement; server-side branch rules remain separate
  • Blockers: None
  • CI/CD issues unrelated to this PR: None observed
  • Dependencies required for testing: Git hooks enabled locally in repo

Workflow Checklist (Required)

  • Branch is up to date with dev
  • PR is from your branch -> dev (no longer directly into main)
  • PR title follows convention (chore:)
  • Plaky feature/bug name included above (N/A)
  • Tagged @Team-Deepiri/support-team
  • Plaky feature moved to "Needs QA" (N/A)

Review Requests

@Team-Deepiri/support-team

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.

1 participant