Skip to content

benchmark diff missing new threshold files#3516

Merged
Lukasa merged 1 commit intoapple:mainfrom
rnro:benchmarks_script_missing_diffs_for_new_files
Feb 23, 2026
Merged

benchmark diff missing new threshold files#3516
Lukasa merged 1 commit intoapple:mainfrom
rnro:benchmarks_script_missing_diffs_for_new_files

Conversation

@rnro
Copy link
Copy Markdown
Contributor

@rnro rnro commented Feb 20, 2026

Motivation

  • When threshold files don't exist yet, thresholds update creates new untracked files. git diff HEAD only shows changes to tracked files, so newly created threshold files were silently omitted from the diff.

Modifications

  • Run git add --intent-to-add . before git diff HEAD so that untracked files are registered in the index and appear in the diff output without staging their contents.

Result

  • The diff output now includes newly created threshold files.

Motivation

* When threshold files don't exist yet, `thresholds update` creates new
untracked files. `git diff HEAD` only shows changes to tracked files,
so newly created threshold files were silently omitted from the diff.

Modifications

* Run `git add --intent-to-add .` before `git diff HEAD` so that
untracked files are registered in the index and appear in the diff
output without staging their contents.

Result

* The diff output now includes newly created threshold files.
@rnro rnro added the semver/none No version bump required. label Feb 20, 2026
@Lukasa Lukasa merged commit 8f6f836 into apple:main Feb 23, 2026
55 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants