Skip to content

fix(ci): align release workflow with ci workflow fixes - #327

Merged
baleen37 merged 1 commit into
mainfrom
fix/ci
Feb 2, 2026
Merged

fix(ci): align release workflow with ci workflow fixes#327
baleen37 merged 1 commit into
mainfrom
fix/ci

Conversation

@baleen37

@baleen37 baleen37 commented Feb 2, 2026

Copy link
Copy Markdown
Owner

Summary

This PR brings .github/workflows/release.yml in line with the fixes made to .github/workflows/ci.yml in commit b7b9bf0.

Changes

  • Install jq via apt-get instead of sergeysova/jq-action@v2 Docker action (jq unavailable after container exits)
  • Remove unsupported cache: false option from bats-core/bats-action@3.0.1
  • Add --no-verify to git commit to skip pre-commit hooks in CI (pre-commit not available in CI environment)
  • Fix timestamp-update.bats test to properly backup/restore update.sh in teardown (was leaving mock content in source file)

Test Plan

  • All BATS tests pass locally: bats tests/
  • Pre-commit hooks pass
  • CI workflow passes on GitHub Actions
  • Release workflow passes on GitHub Actions (after merge)

Generated with Claude Code

Summary by CodeRabbit

  • Tests

    • Enhanced auto-updater test procedures with improved backup and restoration handling for better test isolation.
  • Chores

    • Updated release workflow infrastructure by modifying dependency installation approach.
    • Adjusted release process commit verification configuration.

- Install jq via apt-get instead of sergeysova/jq-action Docker action
- Remove unsupported cache option from bats-core/bats-action
- Add --no-verify to git commit to skip pre-commit hooks in CI
- Fix timestamp-update.bats to backup/restore update.sh in teardown

This brings release.yml in line with the fixes made to ci.yml in commit b7b9bf0.

Co-Authored-By: Claude <noreply@anthropic.com>
@baleen37
baleen37 enabled auto-merge (squash) February 2, 2026 06:56
@coderabbitai

coderabbitai Bot commented Feb 2, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR updates the release workflow to change jq installation from an action to apt-get, removes bats cache configuration, and adds --no-verify flags to git commits. It also enhances test file robustness by implementing backup and restore handling for update.sh.

Changes

Cohort / File(s) Summary
Release Workflow Configuration
.github/workflows/release.yml
Changed jq installation method from sergeysova/jq-action@v2 to apt-get package manager; removed cache configuration from bats setup step; added --no-verify flag to two git commit commands to bypass pre-commit hooks during release operations.
Test Backup/Restore Handling
plugins/auto-updater/tests/timestamp-update.bats
Introduced backup and restoration logic for update.sh file (mirroring existing check.sh handling); added ORIGINAL_UPDATE variable; updated teardown to restore both scripts; minor formatting adjustment to set_last_check_time local variable declaration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • #269: Modifies release.yml git commit steps to add pre-commit hook bypass flags, directly related to the --no-verify additions.
  • #306: Updates timestamp-update.bats with backup/restore handling patterns for updater scripts, following the same test preservation approach.
  • #164: Changes jq installation method in release.yml workflow, directly related to the apt-get installation switch.

Poem

🐰 With workflows refined and tests made strong,
We backup scripts where they belong,
No pre-commit hooks to slow us down,
Fresh jq installed throughout the town! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@baleen37
baleen37 merged commit 6f48dfa into main Feb 2, 2026
1 of 2 checks passed
@baleen37
baleen37 deleted the fix/ci branch February 2, 2026 06:57
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.

1 participant