Skip to content

Feat(cicd_bot): Document and enable the min_intervals plan option #4901

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

erindru
Copy link
Collaborator

@erindru erindru commented Jul 4, 2025

Closes #4069

This PR:

  • Adds documentation for the --min-intervals option which can also be used from the CLI
  • Exposes this option in the CI/CD bot as pr_min_intervals (to be used with default_pr_start)

Note that I called it pr_min_intervals to align with the other pr_* options (pr_environment_name and pr_include_unmodified)

This achieves #4069 by specifying the following config:

cicd_bot:
  default_pr_start: '1 week ago'
  pr_min_intervals: 1

I decided not to introduce the suggested default_pr_relative_start parameter because default_pr_start already supported relative dates.

Instead, I made the "1" in this configurable (as the pr_min_intervals option):

If the computed start/end range does not cover a complete interval, then move start further back until 1 interval is loaded

The logic to give every model its own start date and ensure it covers --min-intervals intervals was added in PR #4780

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.

Add relative start to plan and add support for it to CI/CD bot
1 participant