Skip to content

Clean up release workflow annotations - #15

Merged
merefield merged 1 commit into
masterfrom
release_workflow_cleanup
Aug 23, 2026
Merged

Clean up release workflow annotations#15
merefield merged 1 commit into
masterfrom
release_workflow_cleanup

Conversation

@merefield

Copy link
Copy Markdown
Owner

Summary

  • replace the Bats setup action with Ubuntu's Bats package in CI and release validation
  • remove an unsupported github-token input
  • avoid four unused helper-library caches that failed to restore under /usr/lib
  • remove the deprecated Node 20 cache action pulled in by the Bats setup action

Why

The v0.2.1 release succeeded, but GitHub annotated the validation job for the invalid input, deprecated runtime, and permission-denied cache restores. Termcourse only uses the Bats executable and does not need bats-support, bats-assert, bats-detik, or bats-file.

Validation

  • workflow YAML parsed successfully
  • make check
  • seven installer integration tests pass with Bats 1.10.0

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved review comments were supplied.

Pull request overview

Updates CI and release validation to install Bats via Ubuntu packages instead of the deprecated setup action.

Changes:

  • Removes the unsupported token input.
  • Eliminates obsolete action dependencies and unused caches.
  • Installs Bats with apt-get.
File summaries
File Description
.github/workflows/ci.yml Installs Bats for CI checks.
.github/workflows/release.yml Installs Bats for release validation.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@merefield
merefield merged commit 6e674bc into master Aug 23, 2026
3 checks passed
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.

2 participants