Repository navigation
Fix wording inconsistencies in workflow prompts and script help text - #543
Conversation
- Fix 'Layout' → 'Lay out' grammar in issue-triage ro and rwx variants - Add missing 'any' value to --conclusion help text in fetch-workflow-logs.py - Clarify contradictory command-permission wording in issue-triage/ro - Clarify contradictory command-permission wording in pr-review/ro Co-authored-by: strawgate <6384545+strawgate@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis pull request updates workflow action configuration files and a Python script to improve clarity and fix grammatical errors. Changes include correcting the verb form "Layout" to "Lay out" in two issue-triage workflow variants, clarifying constraint language in read-only workflows by distinguishing between arbitrary and explicitly allowed commands (git operations and helper scripts), tightening documentation about available tools, and adding the "any" value to the 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
Summary
This PR fixes wording inconsistencies in workflow prompts and script help text so instructions match actual allowed behavior.
What changed
Layout→Lay out.scripts/fetch-workflow-logs.pyusage text to include the supportedanyvalue for--conclusion.issue-triage/ro: changedexecute commandstoexecute arbitrary commands, and clarified that only tools/commands listed in(allowed_tools)are available.pr-review/ro: changedexecute commandstoexecute arbitrary commands, and clarified that explicitly listed review helper scripts and read-only git commands are allowed.Context