allow for configuring pr preview default start and min intervals separately#5824
allow for configuring pr preview default start and min intervals separately#5824gjesse wants to merge 3 commits into
Conversation
Signed-off-by: Jesse Hodges <hodges.jesse@gmail.com>
|
I am adding to this to also fix #5789, as they are very closely related |
Signed-off-by: Jesse Hodges <hodges.jesse@gmail.com>
8b75757 to
efa54fa
Compare
|
@gjesse Thank for the PR! A couple of things:
|
glad to help! great catch on that one ;) |
|
@gjesse Can you look into |
…rt=None and using implicit model selection Signed-off-by: Jesse Hodges <hodges.jesse@gmail.com>
09d91b1 to
9591144
Compare
Summary
This updates SQLMesh PR preview planning so forward-only preview intervals can be controlled independently from the regular PR backfill window.
The GitHub bot now supports separate preview defaults for forward-only changes, allowing PR environments to keep broad default backfills where needed without forcing large preview builds for massive forward-only models.
This also fixes a related planning issue where preview fallback dates could accidentally constrain explicitly selected non-preview backfills when preview models were present. Explicit
backfill_modelsselections now keep their normal model-derived interval behavior, while preview fallback starts are only applied when SQLMesh is implicitly selecting the backfill scope.Changes
default_pr_preview_startpr_preview_min_intervalsdefault_pr_start.Validation
Checklist
make styleand fixed any issuesmake fast-test)git commit -s) per the DCO