Skip to content

Update MSRV to 1.89.0 to match AtCoder - #62

Merged
statiolake merged 4 commits into
masterfrom
copilot/update-rust-version-ci
Feb 6, 2026
Merged

Update MSRV to 1.89.0 to match AtCoder#62
statiolake merged 4 commits into
masterfrom
copilot/update-rust-version-ci

Conversation

Copilot AI commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

Update Rust version to 1.89.0 for AtCoder compatibility

  • Update CI workflow (.github/workflows/ci.yml) to use Rust 1.89.0 instead of 1.70.0
  • Update rust-version in proconio/Cargo.toml from 1.70.0 to 1.89.0
  • Update UI test version check in proconio/tests/ui.rs from 1.70.0 to 1.89.0
  • Update related comments in Cargo.toml about MSRV requirements
  • Remove unnecessary winapi-util override (required only Rust 1.71+, now MSRV is 1.89.0)
  • Remove backtrace version pin (no longer needed with MSRV 1.89.0)
  • Run CI tests to validate changes (cargo build, test, clippy, and fmt all pass)
Original prompt

This section details on the original issue you should resolve

<issue_title>Fix CI errors</issue_title>
<issue_description>Rust 1.89.0 is used in AtCoder now. We need to update it in the CI too.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 5, 2026 14:18
Co-authored-by: statiolake <20490597+statiolake@users.noreply.github.com>
Co-authored-by: statiolake <20490597+statiolake@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI errors by updating Rust version to 1.89.0 Update MSRV to 1.89.0 to match AtCoder Feb 5, 2026
Copilot AI requested a review from statiolake February 5, 2026 14:24
Comment thread proconio/Cargo.toml Outdated
# depends on it, and backtrace 0.3.75+ requires a newer Rust version than our
# MSRV (1.70.0)
# MSRV (1.89.0)
backtrace = "=0.3.74"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@copilot Do we still need to pin backtrace? If it passes without pinning, let's remove that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed in 281f7b2. Tests pass successfully with the latest backtrace v0.3.76.

Co-authored-by: statiolake <20490597+statiolake@users.noreply.github.com>
Copilot AI requested a review from statiolake February 5, 2026 14:28
@statiolake
statiolake marked this pull request as ready for review February 6, 2026 15:42
@statiolake
statiolake merged commit ad32c8e into master Feb 6, 2026
32 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.

Fix CI errors

2 participants