Dataset refresh: week 30 candidates from microsoft/BCApps - #743
Draft
Groenbech96 wants to merge 1 commit into
Draft
Dataset refresh: week 30 candidates from microsoft/BCApps#743Groenbech96 wants to merge 1 commit into
Groenbech96 wants to merge 1 commit into
Conversation
Groenbech96
pushed a commit
that referenced
this pull request
Jul 27, 2026
The weekly refresh named its branch/PR after the ISO week the job ran in, but collected a trailing 7-day window ending at the job start time. Those two never describe the same period: the Monday week-30 run opened "week 30" while its contents were week 29 merges (#743 shipped BCApps#9316 / #9313, merged 2026-07-13). The 7-day lookback was correct - those PRs were genuinely inside it - the label was not, so in-window candidates read as stale on review. The trailing window also has fuzzy edges: the bound is date-only and open-ended, so PRs merged between midnight and the job start land in this batch and in the next one. Resolve the window as a full ISO week (Mon 00:00:00Z - Sun 23:59:59Z) via a new Get-IsoWeekWindow helper, default to the last completed week, and derive the branch, PR title and search range from that one window, so "week N" always means "merged during week N". The merge window is also written into the PR body so a candidate's age is explainable at review time. Note: this replaces the since-days dispatch input with weeks-ago. since-days was not wrong, but a trailing N-day window cannot carry an exact week label; Collect-And-Screen.ps1 still accepts -MergedSince/-MergedUntil for arbitrary ranges. Fixes AB#644112 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated dataset refresh: bug-fix candidates collected from
microsoft/BCAppsfor ISO week 30.✅ Included (passed validation)
These entries passed
dataset-validation(build + FAIL_TO_PASS transition) and are included in this PR.microsoft__BCApps-9316(PR #9316) — 🔬 validation❌ Excluded (failed / incomplete validation — documented, not included)
These entries did not pass validation and are not part of this PR's changes; listed for the record.
microsoft__BCApps-9404(PR #9404) — 🔬 validation — failuremicrosoft__BCApps-9313(PR #9313) — 🔬 validation — failure