+
Skip to content

infra: Autoclose stale PRs #1177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
May 28, 2025
Merged

infra: Autoclose stale PRs #1177

merged 3 commits into from
May 28, 2025

Conversation

nicu1989
Copy link
Member

Close PRs older than 30 days.

Related: #1029

@nicu1989 nicu1989 requested a review from Copilot May 28, 2025 07:57
Copilot
Copilot AI reviewed May 28, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new step to automatically close stale pull requests after 30 days of inactivity. The changes include adding a new permission for pull-requests, configuring the actions/stale@v9 action with PR-specific parameters, and adding a comment notification when a PR is closed.

Comments suppressed due to low confidence (1)

.github/workflows/docs-cleanup.yml:24

  • [nitpick] Consider renaming the workflow file or updating its identifier to reflect its updated responsibility (closing stale PRs) rather than just cleaning up docs.
jobs:

@github-actions GitHub Actions
Copy link

The created documentation from the pull request is available at: docu-html

@nicu1989
infra: Autoclose stale PRs
4d17039 
Close PRs older than 30 days.

Signed-off-by: Nicolae Dicu <Nicolae.Dicu.ext@qorix.ai>
@nicu1989 nicu1989 force-pushed the nicu1989_autoclose_stalePR branch from 4333cc6 to 4d17039 Compare May 28, 2025 08:08
AlexanderLanin
AlexanderLanin reviewed May 28, 2025
View reviewed changes
AlexanderLanin
AlexanderLanin reviewed May 28, 2025
View reviewed changes
@AlexanderLanin
Copy link
Member

I wanna look at current PRs before merging this, just a minute....

@AlexanderLanin
Copy link
Member

gh pr list --state open --search "updated:<2025-05-01"

Showing 10 of 10 pull requests in eclipse-score/score that match your search

ID TITLE BRANCH CREATED AT
#890 docs: Added Daal Feature request(#864) 864-feature_request-for-daal about 1 month ago
#874 Api management stakeholder requirements wmader23:api_management_feature_request about 1 month ago
#742 Persistence Features Proposition by Continental on 21.02.2025 guysagnes:conti_IIP-229738 about 2 months ago
#740 docs: added log level guidelines qor-lb-logging-knowledge-base about 2 months ago
#734 This is just a test nradakovic:nira_test_env about 2 months ago
#264 Feature Request: Persistency File Access qorix-group:ulhu_persistency_file_access about 3 months ago
#263 DR: multi-repo setup AlexanderLanin:docs-multirepo about 3 months ago
#220 persistency: Feature request for File Access ulhu_persistency_file_access about 4 months ago
#217 docs: feature flag handling MaximilianSoerenPollak:MaximilianSoerenPollak-packaging-concept about 4 months ago
#192 Update structure to reflect module repository update_directory_layout about 4 months ago

@AlexanderLanin
Copy link
Member

I'm not sure about the latest two. How about a warning at 30 and close at 40?

@nicu1989
Copy link
Member Author

I'm not sure about the latest two. How about a warning at 30 and close at 40?

Will do, I will mark the PRs as stale after 30 days and close after 10 more days of being stall.

nicu1989 added 2 commits May 28, 2025 12:42
@nicu1989
infra: Update after review
5cd769e 
Add extra 10 days between PR is marked as stale and closure.
Aplly also for drafts.

Signed-off-by: Nicolae Dicu <Nicolae.Dicu.ext@qorix.ai>
@nicu1989
infra: Add stale PR message
83b7ff4 
Add a message warning about PR closure.

Signed-off-by: Nicolae Dicu <Nicolae.Dicu.ext@qorix.ai>
@nicu1989 nicu1989 force-pushed the nicu1989_autoclose_stalePR branch from 68be9a1 to 83b7ff4 Compare May 28, 2025 10:56
@AlexanderLanin AlexanderLanin merged commit 67b3d0c into eclipse-score:main May 28, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaximilianSoerenPollak MaximilianSoerenPollak MaximilianSoerenPollak left review comments

Copilot code review Copilot Copilot left review comments

@AlexanderLanin AlexanderLanin AlexanderLanin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nicu1989 @AlexanderLanin @MaximilianSoerenPollak
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载