Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Print stable Rust version in actions runner #347

Merged
merged 1 commit into from
Aug 15, 2024
Merged

Print stable Rust version in actions runner #347

merged 1 commit into from
Aug 15, 2024

Conversation

jpculp
Copy link
Contributor

@jpculp jpculp commented Aug 2, 2024

Description of changes:

Rather than removing the pin entirely, if set to default stable the actions runner will give us useful details about the toolchain.

Example output:

info: using existing install for 'stable-x86_64-unknown-linux-gnu'
info: default toolchain set to 'stable-x86_64-unknown-linux-gnu'

  stable-x86_64-unknown-linux-gnu unchanged - rustc 1.80.0 (051478957 2024-07-21)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jpculp jpculp requested review from rpkelly and gthao313 August 2, 2024 20:55
@jpculp jpculp merged commit 1afe110 into awslabs:develop Aug 15, 2024
2 checks passed
@jpculp jpculp deleted the print-rustc branch August 15, 2024 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgsharm mgsharm mgsharm approved these changes

@gthao313 gthao313 gthao313 approved these changes

@rpkelly rpkelly Awaiting requested review from rpkelly

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jpculp @gthao313 @mgsharm