Skip to content

Commit 94dc896

Browse files
feat: add analysis section to plot-prepare comments
- Why the spec makes sense - Spec-ID reasoning - Similar existing specs comparison
1 parent bd8ca28 commit 94dc896

File tree

1 file changed

+15
-2
lines changed

1 file changed

+15
-2
lines changed

.github/workflows/plot-prepare.yml

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,15 +144,28 @@ jobs:
144144
gh issue edit ${{ github.event.issue.number }} --title "[{spec-id}] {original title without 'Plot: ' prefix}"
145145
```
146146
147-
8. **Post comment with spec content:**
148-
Use this format (read the actual spec.md content you created):
147+
8. **Post comment with analysis and spec content:**
148+
Use this format (include your analysis):
149149
```bash
150150
gh issue comment ${{ github.event.issue.number }} --body "## ✅ Spec Ready: \`{spec-id}\`
151151
152152
**Branch:** \`plot/{spec-id}\`
153153
154154
---
155155
156+
### Analysis
157+
158+
**Why this spec makes sense:**
159+
{1-2 sentences explaining why this plot type is valuable and what use cases it serves}
160+
161+
**Spec-ID reasoning:**
162+
{Explain how you derived the spec-id: what the type/variant/modifier mean}
163+
164+
**Similar existing specs:**
165+
{List any related specs in the repo and explain how this one differs. If none exist, say 'None - this is a new plot category'}
166+
167+
---
168+
156169
### spec.md
157170
158171
{paste full content of spec.md here}

0 commit comments

Comments
 (0)