Skip to content

Conversation

@erik-bershel
Copy link
Contributor

Reverts #13044

Copilot AI review requested due to automatic review settings September 24, 2025 11:57
Copy link
Contributor

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 reverts a previous change that pinned Rust to version 1.89.0, returning to using the stable channel across all operating systems. The change addresses a resolved breaking change that was introduced in Rust 1.90.0.

  • Removes explicit Rust version pinning from 1.89.0
  • Returns to using the stable toolchain channel
  • Updates all three platform-specific installation scripts

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
images/windows/scripts/build/Install-Rust.ps1 Updates Windows Rust installation to use stable toolchain instead of 1.89.0
images/ubuntu/scripts/build/install-rust.sh Updates Ubuntu Rust installation to use stable toolchain instead of 1.89.0
images/macos/scripts/build/install-rust.sh Updates macOS Rust installation to use stable toolchain instead of 1.89.0

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

@erik-bershel erik-bershel merged commit 69c94f5 into main Sep 24, 2025
24 checks passed
@erik-bershel erik-bershel deleted the revert-13044-all-os-pin-rust-to-1.89.0 branch September 24, 2025 17:32
dwydler pushed a commit to dwydler/runner-images-hetzner-cloud that referenced this pull request Sep 26, 2025
@spookyvision
Copy link

spookyvision commented Sep 30, 2025

Copilot reviewed 3 out of 3 changed files in this pull request

slight correction here - Copilot didn't "review" anything, it merely extruded word salad superficially resembling a code review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants