π Fix mission generator quality issues#483
Conversation
- Strip PR template boilerplate (DCO checklists, "Note on DCO" blocks, GitHub asset URLs, "Checklist:" sections) from resolution text - Vary step templates by mission type: features get "Check current setup" and "Verify feature works" instead of "Look for errors" and "Confirm error is gone" - Remove "The root cause is:" prefix from resolution summaries - Add truncateAtSentenceBoundary() to avoid mid-sentence truncation - Clean GitHub checklist markup from both cleanText() and stripPRTemplate() - Feature descriptions say "Requested by N+ users" instead of "This issue affects N+ users" - Default MAX_COPILOT_ISSUES from 10 to 5 for quality control Signed-off-by: Andrew Anderson <andy@clubanderson.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome to KubeStellar! π Thank you for submitting this Pull Request. Before your PR can be merged, please ensure: β
DCO Sign-off - All commits must be signed off with β
PR Title - Must start with an emoji: β¨ (feature), π (bug fix), π (docs), π± (infra/tests), Getting Started with KubeStellar:
Contributor Resources:
π Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback. Whether you're using KubeStellar yourself or know someone who could benefit from multi-cluster Kubernetes: π Take our Multi-Cluster Survey - Share your use cases and help shape our direction! A maintainer will review your PR soon. Feel free to ask questions in the comments or on Slack! |
|
π Thank you for your contribution! Your PR has been successfully merged. π Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback - nothing else. Whether you're using KubeStellar yourself or know organizations that could benefit from multi-cluster Kubernetes, we need your help: π Take our Multi-Cluster Survey - Share your use cases and help shape our direction! π£οΈ Spread the word - Tell colleagues, write blog posts, present at meetups π¬ Share feedback on Slack #kubestellar-dev Every adopter story helps us prioritize what matters most. Thank you for being part of the KubeStellar community! |
Summary
truncateAtSentenceBoundary()to avoid mid-sentence truncationMAX_COPILOT_ISSUESfrom 10 to 5 for quality control batchesTest plan
node --check scripts/generate-cncf-missions.mjspassesDRY_RUN=true TARGET_PROJECTS=argo MAX_COPILOT_ISSUES=2completes