Skip to content

fix(ci): add apt-get update before install in release-snapshot job - #205

Merged
TMHSDigital merged 1 commit into
TMHSDigital:mainfrom
HarshRajSinghania:fix/ci-apt-update-release-snapshot
Nov 9, 2026
Merged

TMHSDigital merged 1 commit into
TMHSDigital:mainfrom
HarshRajSinghania:fix/ci-apt-update-release-snapshot

Conversation

@HarshRajSinghania

Copy link
Copy Markdown
Contributor

Summary

Adds sudo apt-get update -qq before the package install in the release-snapshot job's "Check completions and man page" step, and includes --no-install-recommends.

This matches the pattern already used in the build job and prevents 404 failures from stale apt indexes on Ubuntu runners.

Motivation

Fixes #203. The Go workflow has been failing on every push to main at this step since the package index became stale, even though other jobs pass. This hides real regressions.

Implementation

  • In .github/workflows/go.yml, updated the step to refresh the package index before installing zsh, fish, and groff.
  • Dropped the >/dev/null redirection so apt errors are visible in logs.
  • Added --no-install-recommends for consistency with the earlier install step.

Testing

  • Diff reviewed against the suggested fix in the issue.
  • The change is minimal and follows existing workflow conventions.
  • Full CI verification requires a push (the failure is environment-dependent on the runner's apt cache).

Closes #203.

The Check completions and man page step was failing with 404s on stale package indexes.
Add apt-get update and --no-install-recommends, matching the pattern used earlier in the workflow.

Fixes TMHSDigital#203
@TMHSDigital

Copy link
Copy Markdown
Owner

Thanks, this is exactly the fix #203 asked for, and dropping the >/dev/null means a future apt failure will be readable in the log. Merging; I'll watch the Go workflow on main to confirm release-snapshot goes green.

@TMHSDigital
TMHSDigital merged commit 61b3ce0 into TMHSDigital:main Nov 9, 2026
2 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.

release-snapshot CI step runs apt-get install without apt-get update

2 participants