Skip to content

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

Merged
erindru merged 1 commit intomainfrom
erin/cicd-bot-min-intervals
Jul 7, 2025
Merged

Feat(cicd_bot): Document and enable the min_intervals plan option#4901
erindru merged 1 commit intomainfrom
erin/cicd-bot-min-intervals

Conversation

@erindru
Copy link
Copy Markdown
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

@erindru erindru merged commit b3bd132 into main Jul 7, 2025
27 checks passed
@erindru erindru deleted the erin/cicd-bot-min-intervals branch July 7, 2025 21:52
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

2 participants