Skip to content

Add stale prs#1053

Open
CsatariGergely wants to merge 3 commits into
kptdev:mainfrom
nokia:add-stale-prs
Open

Add stale prs#1053
CsatariGergely wants to merge 3 commits into
kptdev:mainfrom
nokia:add-stale-prs

Conversation

@CsatariGergely

@CsatariGergely CsatariGergely commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@netlify

netlify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch failed. Why did it fail? →

Name Link
🔨 Latest commit 8ae701b
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a339674203c3a0008c49169

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Not ready to approve

The new documentation contains typos/grammar issues and includes potentially misleading wording about stale PR closure (implying automation without evidence).

Pull request overview

Updates the contributor guide’s PR review/merge expectations by documenting the agreed approval requirements and introducing a “stale PR” policy.

Changes:

  • Reflowed/clarified the “Reviews of Your PR” checklist formatting.
  • Added an explicit requirement for approvals from both the contributors and maintainers teams before merging.
  • Added guidance for handling “stale PRs” (closure after a period of inactivity).
File summaries
File Description
docs/content/en/docs/12_contributing/code-contribution/_index.md Documents new PR approval rules and adds a stale PR policy section in the contribution guide.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 3

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.

Comment thread docs/content/en/docs/12_contributing/code-contribution/_index.md Outdated
Comment thread docs/content/en/docs/12_contributing/code-contribution/_index.md Outdated
Comment thread docs/content/en/docs/12_contributing/code-contribution/_index.md Outdated
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 17, 2026
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Assisted-by: GitHub Copilot: Claude Haiku 4.5

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants