Second round SDK CI/CD sync + exceed by jeremy · Pull Request #167 · basecamp/basecamp-cli
AI review requested due to automatic review settings
February 28, 2026 10:48Copilot AI review requested due to automatic review settings
February 28, 2026 10:55CI is the safety gate — auto-merging action updates is circular since a broken update approves itself. Invert the policy to auto-merge patch+minor for all ecosystems except github_actions. The ecosystem name uses underscore (github_actions) because fetch-metadata derives it from the branch name, not dependabot.yml.
Pin schedule to 06:00 America/Chicago for predictable update timing. Set open-pull-requests-limit to 10 for both gomod and github-actions.
go-mod-tidy runs on commit to catch untidy modules before CI. go-test-short runs on push only to avoid slowing down commits. Uses files pattern instead of types: [go] so go.mod/go.sum-only changes also trigger the hooks. Adds default_install_hook_types so pre-commit install sets up both pre-commit and pre-push hooks.
Copilot AI review requested due to automatic review settings
February 28, 2026 11:01Add non-mutating tidy-check Makefile target that backs up go.mod/go.sum, runs go mod tidy, then verifies no diff — restoring originals on failure so make check never leaves the working tree dirty. Add both provenance-check and tidy-check to the check gate. Add tidy-check step to test.yml before unit tests. Fix pre-existing untidy go.mod (pflag was indirect but is directly imported).
jeremy
deleted the
ci/sdk-sync-round2
branch
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