Skip to content

Clarify script freshness checks - #27

Merged
codeforester merged 1 commit into
masterfrom
enhancement/13-20260618-script-freshness-contract
Jun 18, 2026
Merged

codeforester merged 1 commit into
masterfrom
enhancement/13-20260618-script-freshness-contract

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • Keep check_script_up_to_date local-ref-only by default and log that freshness contract.
  • Add --fetch to refresh upstream state before comparing when callers want a live remote check.
  • Document the freshness behavior and add stale-ref/fetch BATS coverage.

Validation

  • RED: bats --filter "check_script_up_to_date (uses local|fetches)" lib/bash/git/tests/lib_git.bats failed before the fix because local-ref messaging and --fetch support were missing.
  • GREEN: bats --filter "check_script_up_to_date (uses local|fetches)" lib/bash/git/tests/lib_git.bats
  • bats lib/bash/git/tests/lib_git.bats
  • shellcheck --severity=error lib/bash/git/lib_git.sh lib/bash/git/tests/lib_git.bats
  • ./tests/validate.sh
  • git diff --check

Fixes #13

@codeforester
codeforester merged commit 9d94170 into master Jun 18, 2026
2 checks passed
@codeforester
codeforester deleted the enhancement/13-20260618-script-freshness-contract branch June 18, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify freshness semantics for check_script_up_to_date

1 participant