Skip to content

Deprecates the probot-scheduler and introduces a custom scheduler - #372

Open
lkbhitesh07 wants to merge 1 commit into
oppia:masterfrom
lkbhitesh07:replace-probot-scheduler
Open

lkbhitesh07 wants to merge 1 commit into
oppia:masterfrom
lkbhitesh07:replace-probot-scheduler

Conversation

@lkbhitesh07

@lkbhitesh07 lkbhitesh07 commented Jun 9, 2026

Copy link
Copy Markdown
Member

Explanation

This PR deprecates the probot-scheduler which is no longer maintained and introduces a custom scheduler. This is part of a bigger migration we are planning where the oppiabot-stale will be merged into oppiabot. The reason why we need a custom scheduler is because the stale checks are scheduled to run every hour and the checks in oppiabot is scheduled to run once every 24 hours, the custom scheduler can now create 2 schedules.

##Proof
The test-app is able to add the comment per the scheduler, I added the schedule for every 2 mins.
Screenshot from 2026-06-13 15-51-44

Checklist

  • I have successfully deployed my own instance of Oppiabot.
    • You can find instructions for doing this here.
  • I have manually tested all the changes made in this PR following the manual tests matrix.

@lkbhitesh07
lkbhitesh07 marked this pull request as ready for review June 13, 2026 10:25
@lkbhitesh07
lkbhitesh07 requested review from a team as code owners June 13, 2026 10:25
@lkbhitesh07
lkbhitesh07 requested review from U8NWXD and removed request for a team June 13, 2026 10:25
@lkbhitesh07

Copy link
Copy Markdown
Member Author

@U8NWXD PTAL!

@U8NWXD U8NWXD left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. please just fix the date in the copyright header. sorry this took me so long!

Comment thread spec/schedulerSpec.js
@@ -0,0 +1,237 @@
// Copyright 2020 The Oppia Authors. All Rights Reserved.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
// Copyright 2020 The Oppia Authors. All Rights Reserved.
// Copyright 2026 The Oppia Authors. All Rights Reserved.

@U8NWXD U8NWXD assigned lkbhitesh07 and unassigned U8NWXD Jul 11, 2026
@oppiabot oppiabot Bot added the PR: LGTM label Jul 11, 2026
@oppiabot

oppiabot Bot commented Jul 11, 2026

Copy link
Copy Markdown

Hi @lkbhitesh07, this PR is ready to be merged. Please address any remaining comments prior to merging, and feel free to ask someone to merge your PR once the CI checks pass and you're happy with it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement scheduler that replaces probot-scheduler which is depreciated.

2 participants