Skip to content

Fix CI pipeline, podman container build#36

Merged
thomasw04 merged 4 commits intomainfrom
fix/ci
Mar 1, 2026
Merged

Fix CI pipeline, podman container build#36
thomasw04 merged 4 commits intomainfrom
fix/ci

Conversation

@xarantolus
Copy link
Contributor

No description provided.

Also restoring MIN_RANGE_SIZE that seems to have been deleted accidentally in 86e35b2
@xarantolus xarantolus marked this pull request as ready for review March 1, 2026 10:22
Copilot AI review requested due to automatic review settings March 1, 2026 10:22
@xarantolus
Copy link
Contributor Author

CI fails because we use the container from the main branch only (see #35), but this should work fine once merged

@xarantolus
Copy link
Contributor Author

Also had to move some code around to fix kani

Copy link

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

Updates the devcontainer configuration and build image to support the CI/container build pipeline changes (notably around container caching/build behavior, likely for Podman compatibility).

Changes:

  • Adjust .devcontainer/devcontainer.json cacheFrom to use a single GHCR image reference.
  • Bump the Rust toolchain version used in the devcontainer Dockerfile.

Reviewed changes

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

File Description
.devcontainer/devcontainer.json Updates registry cache source image reference used by devcontainer builds.
.devcontainer/Dockerfile Changes the pinned Rust toolchain version installed via rustup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xarantolus xarantolus requested a review from thomasw04 March 1, 2026 12:15
@thomasw04 thomasw04 merged commit bafd728 into main Mar 1, 2026
4 of 12 checks passed
@thomasw04 thomasw04 deleted the fix/ci branch March 1, 2026 16:22
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.

3 participants