You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: .github/workflows/claude-fix-issue.yml
+4-2Lines changed: 4 additions & 2 deletions
Original file line number
Diff line number
Diff line change
@@ -121,15 +121,17 @@ jobs:
121
121
- src/Reflection/ - Reflection layer
122
122
123
123
Read CLAUDE.md for important guidelines about the codebase architecture and common patterns.
124
+
In case your fix changes more than 75 lines in src/ try a different fix.
124
125
125
126
## Step 3: Verify the fix
126
127
127
128
1. Run the regression test to confirm it passes now
128
129
2. Run the full test suite: make tests
129
130
3. Run PHPStan self-analysis: make phpstan
130
131
4. Fix any failures that come up
131
-
5. Run make cs-fix to fix any coding standard violations
132
-
6. Run make name-collision and fix violations - add different tests in unique namespaces. If the function and class declarations are exactly the same, you can reuse them across files instead of duplicating them.
132
+
5. Verify all src/ changes made are really required to make the tests pass. Revert/Remove changes which are unrelated.
133
+
6. Run make cs-fix to fix any coding standard violations
134
+
7. Run make name-collision and fix violations - add different tests in unique namespaces. If the function and class declarations are exactly the same, you can reuse them across files instead of duplicating them.
133
135
134
136
Do not create a branch, push, or create a PR - this will be handled automatically.
0 commit comments