Skip to content

Conversation

zeriyoshi
Copy link
Collaborator

No description provided.

@Copilot Copilot AI review requested due to automatic review settings August 28, 2025 07:08
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades the Debian base image from bookworm (Debian 12) to trixie (Debian 13) across the Docker build configuration and CI workflows.

  • Updates Docker base image tag from 8.4-cli-bookworm to 8.4-cli-trixie
  • Simplifies clang package specification from clang-19 to generic clang
  • Updates CI workflow matrix configurations to use trixie instead of bookworm

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
Dockerfile Updates base image tag and clang package specification
.github/workflows/coverage.yaml Updates distro matrix from bookworm to trixie
.github/workflows/ci.yaml Updates distro matrix from bookworm to trixie for both job configurations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"clang" \
"lcov" "gzip" \
"vim" \
&& update-alternatives --install "/usr/bin/clang" clang "/usr/bin/clang-19" 100 \
Copy link
Preview

Copilot AI Aug 28, 2025

Choose a reason for hiding this comment

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

The update-alternatives command still references /usr/bin/clang-19 but the package installation was changed to generic clang. This will fail if clang-19 is not available in trixie or if the generic clang package installs a different version.

Copilot uses AI. Check for mistakes.

Copy link

Code Metrics Report

main (b0fdfb5) #35 (f970324) +/-
Coverage 61.7% 61.7% 0.0%
Details
  |           | main (b0fdfb5) | #35 (f970324) | +/-  |
  |-----------|----------------|---------------|------|
  | Coverage  |          61.7% |         61.7% | 0.0% |
  |   Files   |              5 |             5 |    0 |
  |   Lines   |           1008 |          1008 |    0 |
  |   Covered |            622 |           622 |    0 |

Reported by octocov

@zeriyoshi zeriyoshi merged commit 65e512d into main Aug 28, 2025
53 checks passed
@zeriyoshi zeriyoshi deleted the trixie branch August 28, 2025 10:17
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