Skip to content

Conversation

@MonaaEid
Copy link
Contributor

@MonaaEid MonaaEid commented Dec 6, 2025

  • Description:

This pull request introduces a new workflow to automatically remind authors of open pull requests that have been inactive for 10 days. The workflow is scheduled to run daily and uses a custom script to identify inactive PRs and post a reminder comment if one hasn't already been posted.

New PR inactivity reminder workflow:

Workflow automation:

  • Added .github/workflows/pr-inactivity-reminder-bot.yml to schedule and run a job that checks for inactive PRs and posts reminders using GitHub Actions.

Bot logic implementation:

  • Created .github/scripts/pr_inactivity_reminder.js to find open PRs with no commit activity for 10 days, check for existing bot comments, and post a standardized reminder if needed. The script handles edge cases such as missing commit info or forked repositories.

Related issue(s):

Fixes #950

Checklist

  • Created PR inactivity reminder bot
  • Updated CHANGELOG.md
  • Documented (Code comments)
  • Tested (unit, integration, logs)

@MonaaEid MonaaEid marked this pull request as ready for review December 6, 2025 17:40
Copilot AI review requested due to automatic review settings December 6, 2025 17:40
Copy link
Contributor

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 pull request introduces an automated workflow to remind authors of pull requests that have been inactive for 10 days. The bot runs daily via GitHub Actions and posts a standardized reminder comment to inactive PRs, helping maintain development momentum and providing support resources.

Key Changes:

  • Automated daily PR inactivity checks with configurable 10-day threshold
  • Smart comment detection to prevent duplicate reminders
  • Fallback mechanisms for fetching commit dates from forked repositories

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
.github/workflows/pr-inactivity-reminder-bot.yml Defines the scheduled workflow that runs daily at 11:00 UTC to check for and comment on inactive PRs
.github/scripts/pr_inactivity_reminder.js Implements the bot logic to fetch PR commit dates, check for existing bot comments, and post reminder messages with Discord and office hours links
CHANGELOG.md Documents the addition of the PR inactivity reminder workflow in the Unreleased section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MonaaEid MonaaEid requested a review from exploreriii December 7, 2025 11:18
@exploreriii
Copy link
Contributor

I am reviewing this currently but we have 4 concurrent bot PRs to review.
Request help to review @MonaaEid @tech0priyanshu @AntonioCeppellini @Adityarya11 if available

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

Hi @MonaaEid excellent work!
Have added some suggestions to get to the final stages 👍
Please change PR status to draft until it is ready to review agian

@MonaaEid MonaaEid marked this pull request as draft December 7, 2025 18:58
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

To assist you, please read:

Thank you for contributing!

From the Hiero Python SDK Team

@MonaaEid MonaaEid force-pushed the feat/950-pr-reminder-bot branch 2 times, most recently from 8d95829 to b57070a Compare December 8, 2025 20:30
Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

@MonaaEid please click the sync fork button in your online repo and then pull it as you are currently not in sync with main

Image

Imagesync fork button

@MonaaEid
Copy link
Contributor Author

MonaaEid commented Dec 8, 2025

@exploreriii my commit won't be discard?

@MonaaEid MonaaEid closed this Dec 8, 2025
@MonaaEid MonaaEid force-pushed the feat/950-pr-reminder-bot branch from b57070a to b7f8b99 Compare December 8, 2025 21:35
Signed-off-by: MonaaEid <monaa_eid@hotmail.com>
Signed-off-by: MonaaEid <monaa_eid@hotmail.com>
@MonaaEid MonaaEid reopened this Dec 8, 2025
@MonaaEid MonaaEid marked this pull request as ready for review December 9, 2025 12:24
Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

Hi @MonaaEid please rebase and happy to approve this

if you have time, please change the harded 'your PR has been inactive for 10 days' to
'your pR has been inactive fro at least 10 days'
or calculate the day more precisely

@exploreriii exploreriii marked this pull request as draft December 9, 2025 18:09
Signed-off-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
Signed-off-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
Added a PR inactivity reminder bot for stale pull requests.

Signed-off-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
@exploreriii exploreriii marked this pull request as ready for review December 9, 2025 22:07
@exploreriii exploreriii merged commit c3462f6 into hiero-ledger:main Dec 9, 2025
14 checks passed
adityashirsatrao007 pushed a commit to adityashirsatrao007/hiero-sdk-python that referenced this pull request Dec 10, 2025
…edger#978)

Signed-off-by: MonaaEid <monaa_eid@hotmail.com>
Signed-off-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
@MonaaEid MonaaEid deleted the feat/950-pr-reminder-bot branch December 11, 2025 14:12
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.

Create inactivity reminder bot for pull requests that are inactive

2 participants