Skip to content

Commit 9b3f639

Browse files
test: proper workflow validation with standard flow
Added PROPER_TEST_FLOW.md documenting correct branch flow: - Step 1: test/proper-workflow-validation → dev - Step 2: dev → main This PR tests that test/* branches: - CANNOT merge directly to main (will be rejected) - CAN merge to dev (this PR) - Follow standard flow through dev After this merges to dev, we'll create dev → main PR to test complete release pipeline with all release gates. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 7897c65 commit 9b3f639

File tree

1 file changed

+67
-0
lines changed

1 file changed

+67
-0
lines changed

PROPER_TEST_FLOW.md

Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
# Proper Workflow Test - Standard Flow
2+
3+
**Date**: 2025-11-07
4+
**Flow**: `test/proper-workflow-validation → dev → main`
5+
**Purpose**: Validate correct branch flow and release gates
6+
7+
---
8+
9+
## ✅ Correct Flow
10+
11+
### Step 1: test/* → dev
12+
- Branch: `test/proper-workflow-validation`
13+
- Target: `dev`
14+
- Expected: PR should be allowed and checks should run
15+
16+
### Step 2: dev → main
17+
- Branch: `dev`
18+
- Target: `main`
19+
- Expected: Release gates should run and pass
20+
21+
---
22+
23+
## What This Tests
24+
25+
### Branch Pattern Enforcement
26+
- ✅ test/* branches CANNOT merge directly to main
27+
- ✅ test/* branches CAN merge to dev
28+
- ✅ dev branch CAN merge to main
29+
- ✅ Proper error messages when flow is violated
30+
31+
### Standard Flow Validation
32+
- ✅ PR checks run on test/* → dev
33+
- ✅ Release gates run on dev → main
34+
- ✅ All quality gates execute properly
35+
- ✅ No workflow bypasses
36+
37+
### Complete Release Pipeline
38+
1. Feature/test development on test/* branch
39+
2. PR and merge to dev
40+
3. Dev accumulates changes
41+
4. Release PR from dev to main
42+
5. Release gates validate production readiness
43+
6. Merge to main (production)
44+
45+
---
46+
47+
## Success Criteria
48+
49+
**PR #1 (test/* → dev):**
50+
- ✅ PR allowed and created
51+
- ✅ Quality checks run
52+
- ✅ Can be merged to dev
53+
54+
**PR #2 (dev → main):**
55+
- ✅ Source branch validation passes (dev is allowed)
56+
- ✅ Production build succeeds
57+
- ✅ Smoke tests pass
58+
- ✅ Security scan runs (informational)
59+
- ✅ Deployment readiness completes
60+
- ✅ Release gate status evaluates correctly
61+
- ✅ Can be merged to main
62+
63+
---
64+
65+
**Created**: 2025-11-07
66+
**Previous PR #8**: Closed (test/* → main not allowed)
67+
**This PR Flow**: test/* → dev → main (correct)

0 commit comments

Comments
 (0)