Skip to content
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.

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

Feat: add different time schedules #55

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

mjpayne8
Copy link

@mjpayne8 mjpayne8 commented Jul 21, 2022

Added functionality to create reoccuring schedules based on minute, day, week and month reoccurance.

Changes include:

  • Changed keys in schedule toml block: every_n_minutes -> every_n_units
  • Additional keys added to schedule block:
    +unit
    +run_days
    +day_of_month
    +window_start
    +window_end
  • Transformation function to transform schedule toml block into the necessary sp_add_jobschedule parameters

Change should support following schedules:
'every x minutes between window_start and window_end'
'every x days at y time'
'every x weeks on specified days of week at y time'
'every x months on specified day of month at y time'

@github-actions
Copy link

github-actions bot commented Jul 21, 2022

Unit Test Results

58 tests  +14   57 ✔️ +14   0s ⏱️ ±0s
  1 suites ±  0     1 💤 ±  0 
  1 files   ±  0     0 ±  0 

Results for commit c1ec61c. ± Comparison against base commit 6ff145f.

♻️ This comment has been updated with latest results.

@mjpayne8 mjpayne8 requested a review from gvee-uk July 21, 2022 13:36
@mjpayne8 mjpayne8 changed the title Feat/add different time schedules Feat: add different time schedules Jul 22, 2022
@mjpayne8 mjpayne8 linked an issue Jul 22, 2022 that may be closed by this pull request
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.

More Job Schedule Options
1 participant