Skip to content

Commit af64998

Browse files
authored
fix(sync): only advance sync SHA pointer when new tools are added (#1037)
1 parent ce8d80b commit af64998

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

‎scripts/sync-tools-repo.py‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1029,8 +1029,10 @@ def run(self, report_file: Optional[Path] = None) -> int:
10291029
file=sys.stderr,
10301030
)
10311031

1032-
# Advance the SHA pointer (same commit as YAML changes)
1033-
if self._pending_new_sha and self.last_sync_sha_file is not None:
1032+
# Only advance the sync pointer when this run actually adds tools.
1033+
# Otherwise we would generate a SHA-only PR and stop reconsidering the
1034+
# same source changes on later runs.
1035+
if self.new_tools and self._pending_new_sha and self.last_sync_sha_file is not None:
10341036
if not self.dry_run:
10351037
self.last_sync_sha_file.write_text(self._pending_new_sha + "\n")
10361038
print(

0 commit comments

Comments
 (0)