Skip to content

updater: skip unnecessary updater steps to reduce execution time#677

Open
sale3 wants to merge 2 commits intomasterfrom
sbojanic/check-commit-hashes-before-validating
Open

updater: skip unnecessary updater steps to reduce execution time#677
sale3 wants to merge 2 commits intomasterfrom
sbojanic/check-commit-hashes-before-validating

Conversation

@sale3
Copy link
Contributor

@sale3 sale3 commented Nov 28, 2025

Description (e.g. "Related to ...", etc.)

Skip all unnecessary updater steps if last validated commit is equal to local and remote HEAD.

Test results: no-op taf repo update comes down from 11 to 0 for test repository.

Closes: #676

Code review checklist (for code reviewer to complete)

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Commit messages are meaningful (see this for details)
  • Tests have been included and/or updated, as appropriate
  • Docstrings have been included and/or updated, as appropriate
  • Changelog has been updated, as needed (see CHANGELOG.md)

@sale3 sale3 requested a review from n-dusan November 28, 2025 21:47
@sale3 sale3 self-assigned this Nov 28, 2025
@sale3 sale3 added the updater label Nov 28, 2025
@sale3 sale3 force-pushed the sbojanic/check-commit-hashes-before-validating branch from 42b8d24 to d26ef9c Compare December 5, 2025 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updater pipeline adjustment

1 participant