Skip to content

Conversation

@jamadeo
Copy link
Collaborator

@jamadeo jamadeo commented Jan 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 6, 2026 17:35
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 updates the Rust toolchain from version 1.88.0 to 1.92 (current stable) and upgrades the rustup package manager from version 1.25.2 to 1.28.2.

Key Changes:

  • Updated Rust toolchain channel from 1.88.0 to 1.92 in the toolchain configuration
  • Upgraded rustup from version 1.25.2 to 1.28.2 across all Rust binary tools
  • Removed the old rustup 1.25.2 package file

Reviewed changes

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

Show a summary per file
File Description
rust-toolchain.toml Updated Rust toolchain channel from 1.88.0 to 1.92
bin/rustup Updated rustup package reference from 1.25.2 to 1.28.2
bin/rustfmt Updated rustup package reference from 1.25.2 to 1.28.2
bin/rustdoc Updated rustup package reference from 1.25.2 to 1.28.2
bin/rustc Updated rustup package reference from 1.25.2 to 1.28.2
bin/rust-lldb Updated rustup package reference from 1.25.2 to 1.28.2
bin/rust-gdbgui Updated rustup package reference from 1.25.2 to 1.28.2
bin/rust-gdb Updated rustup package reference from 1.25.2 to 1.28.2
bin/rust-analyzer Updated rustup package reference from 1.25.2 to 1.28.2
bin/rls Updated rustup package reference from 1.25.2 to 1.28.2
bin/clippy-driver Updated rustup package reference from 1.25.2 to 1.28.2
bin/cargo-miri Updated rustup package reference from 1.25.2 to 1.28.2
bin/cargo-fmt Updated rustup package reference from 1.25.2 to 1.28.2
bin/cargo-clippy Updated rustup package reference from 1.25.2 to 1.28.2
bin/cargo Updated rustup package reference from 1.25.2 to 1.28.2
bin/.rustup-1.25.2.pkg Removed old rustup package file

Copilot AI review requested due to automatic review settings January 6, 2026 18:31
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

Copilot reviewed 12 out of 27 changed files in this pull request and generated 2 comments.

Self: Sized,
{
let model_strings: Vec<String> = vec![
let model_strings: Vec<String> = [
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

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

Type mismatch: assigning an array directly to a Vec variable. The array should be followed by .to_vec() or the type annotation should be removed to allow type inference.

Copilot uses AI. Check for mistakes.
#[test]
fn test_valid_conversation() {
let all_messages = vec![
let all_messages = [
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

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

Type mismatch: assigning an array directly to a Vec variable. The array should be followed by .to_vec() or the type annotation should be removed to allow type inference.

Copilot uses AI. Check for mistakes.
))
}

#[derive(Serialize, Deserialize, Debug)]
Copy link
Collaborator

Choose a reason for hiding this comment

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

were these just unused?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah seems so, and I'm not sure why it wasn't flagged by the build before, but I'm guessing some improvements were made between 1.88 and 1.92 to finding dead code

@jamadeo jamadeo merged commit fb1d3df into main Jan 6, 2026
20 checks passed
@jamadeo jamadeo deleted the rust-toolchain-1.92 branch January 6, 2026 19:13
zanesq added a commit that referenced this pull request Jan 7, 2026
* 'main' of github.com:block/goose:
  Claude 3.7 is out. we had some harcoded stuff (#6197)
  Release 1.19.0
  chore: upgrade to node v24 as engine (#6361)
  chore(deps): bump rsa from 0.9.9 to 0.9.10 (#6358)
  Bump rust toolchain to 1.92 (current stable) (#6356)
  Hide advanced recipe options under expandable content (#6021)
  fix: use .config/agents (plural) for skills directory (#6357)
  fix: prevent KaTeX from treating underscores as subscripts in plain text (#6242)
  fix: make goose review PRs more like goose contributors do (#6240)
  fix : preserve provider engine type when editing custom providers (#6106)
  feat(providers): add retry for model fetching (#6347)
  allow goose issue solver to react to activation comments (#6239)
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