Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Scripts for closing pull requests for Hacktoberfest#11587

Merged
cclauss merged 3 commits intoTheAlgorithms:masterTheAlgorithms/Python:masterfrom
cclauss:scripts_for_closing_pull_requestscclauss/Python:scripts_for_closing_pull_requestsCopy head branch name to clipboard
Sep 30, 2024
Merged

Scripts for closing pull requests for Hacktoberfest#11587
cclauss merged 3 commits intoTheAlgorithms:masterTheAlgorithms/Python:masterfrom
cclauss:scripts_for_closing_pull_requestscclauss/Python:scripts_for_closing_pull_requestsCopy head branch name to clipboard

Conversation

@cclauss
Copy link
Member

@cclauss cclauss commented Sep 30, 2024

Each year we prepare for Hacktoberfest by clearing out old pull requests to simplify things for project maintainers.

We had 211 open pull requests; after running these scripts, we have 126. ~40% of our Open PRs were closed.

@MaximSmolskiy Your review, please.

Copy link
Contributor

@tianyizheng02 tianyizheng02 left a comment

Choose a reason for hiding this comment

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

Shouldn't we increase the PR limit to something higher than 200? We regularly have open PR counts larger than that.

Copy link
Contributor

@tianyizheng02 tianyizheng02 left a comment

Choose a reason for hiding this comment

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

Admittedly I'm not great with bash scripting, but the scripts look right to me. I trust that they work as intended if these are the same scripts that you just used clean up our open PRs, which appears to have been successful.

scripts/close_pull_requests_with_awaiting_changes.sh Outdated Show resolved Hide resolved
@cclauss cclauss merged commit a9ca110 into TheAlgorithms:master Sep 30, 2024
@algorithms-keeper algorithms-keeper bot removed the awaiting reviews This PR is ready to be reviewed label Sep 30, 2024
@cclauss cclauss deleted the scripts_for_closing_pull_requests branch September 30, 2024 10:49
@tjgurwara99
Copy link
Member

Just passing by here since I got assigned in this so the following is a question and maybe a suggestion... - is it worth doing this in a workflow_dispatch based CI action? https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#workflow_dispatch

These seem to be repeatable tasks and would be easier to do them directly from the GitHub UI for maintainers.

@cclauss
Copy link
Member Author

cclauss commented Sep 30, 2024

I hesitate to automate this too much because we run it rarely and closing other people's work is something we should do with some deliberation and not too much automation.

@isidroas isidroas mentioned this pull request Jan 25, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Morty Proxy This is a proxified and sanitized view of the page, visit original site.