chore: remove obsolete workflows and align stale messaging#402
Conversation
Drop outdated development, docs, and PR review workflows that are no longer used by this repository. Update stale issue/PR messages to match the configured 60-day stale and 30-day close thresholds. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Warning
|
| Layer / File(s) | Summary |
|---|---|
Workflow Removals .github/workflows/agentready-dev-issue-pr.yml, .github/workflows/agentready-dev.yml, .github/workflows/pr-review.yml, .github/workflows/docs.yml |
Four workflows removed: agentready-dev-issue-pr.yml (796 lines) for automating issues to draft PRs, agentready-dev.yml (355 lines) for handling @agentready-dev mentions, pr-review.yml (83 lines) for Claude-based PR reviews, and docs.yml (40 lines) for lychee-based link checking on documentation PRs. |
Stale Issue Configuration Update .github/workflows/stale-issues.yml |
days-before-stale increased from 21 to 60 days; days-before-close increased from 7 to 30 days, extending total inactivity window to 90 days. Stale and closure messages updated to reflect new 60-day and 90-day thresholds for both issues and PRs. |
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title accurately summarizes the main changes: removing obsolete workflows and updating stale messaging thresholds. |
| Description check | ✅ Passed | The description clearly explains what was removed and what was updated, relates directly to the changeset, and provides context. |
| Linked Issues check | ✅ Passed | The PR addresses issue #397 by removing docs.yml workflow and updates stale-issues.yml messaging to reflect configured thresholds as intended. |
| Out of Scope Changes check | ✅ Passed | All changes are in scope: four workflow removals align with the cleanup objective, and stale-issues.yml updates align with messaging alignment objective. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
chore/remove-obsolete-workflows
Tip
💬 Introducing Slack Agent: The best way for teams to turn conversations into code.
Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
- Generate code and open pull requests
- Plan features and break down work
- Investigate incidents and troubleshoot customer tickets together
- Automate recurring tasks and respond to alerts with triggers
- Summarize progress and report instantly
Built for teams:
- Shared memory across your entire org—no repeating context
- Per-thread sandboxes to safely plan and execute work
- Governance built-in—scoped access, auditability, and budget controls
One agent for your entire SDLC. Right inside Slack.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/stale-issues.yml:
- Line 21: The inline comment next to the days-before-stale setting is
incorrect; update the comment to match the configured value by changing the
comment for days-before-stale from “30 days before 90-day close” to “60 days
before 90-day close” (referencing the days-before-stale key in the workflow).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Enterprise
Run ID: c70e4801-08f0-4f55-ae5f-be39b3e76b03
📒 Files selected for processing (5)
.github/workflows/agentready-dev-issue-pr.yml.github/workflows/agentready-dev.yml.github/workflows/docs.yml.github/workflows/pr-review.yml.github/workflows/stale-issues.yml
💤 Files with no reviewable changes (4)
- .github/workflows/docs.yml
- .github/workflows/pr-review.yml
- .github/workflows/agentready-dev.yml
- .github/workflows/agentready-dev-issue-pr.yml
📈 Test Coverage Report
Coverage calculated from unit tests only |
jwm4
left a comment
There was a problem hiding this comment.
All changes look good to me. The one complaint from CodeRabbit seems to be wrong (60 is 30 days from 90). I will merge this now.
|
🎉 This PR is included in version 2.35.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
agentready-dev-issue-pr.yml,agentready-dev.yml,pr-review.yml, anddocs.yml.stale-issues.ymlto match configured thresholds: 60 days stale, 30 days to close (90 days total).Related issues
Test plan
.github/workflows/.Made with Cursor