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.

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

Output the rust version for each build. #288

Merged
merged 1 commit into from
Jun 3, 2021
Merged

Conversation

markmandel
Copy link
Member

This makes it easier to determine if issues are related to which Rust version is being run in CI.

Work on #281

@markmandel markmandel added kind/cleanup Refactoring code, fixing up documentation, etc area/build-tools Development tooling. labels Jun 2, 2021
@google-cla google-cla bot added the cla: yes label Jun 2, 2021
@quilkin-bot

This comment has been minimized.

@quilkin-bot

This comment has been minimized.

This makes it easier to determine if issues are related to which Rust
version is being run in CI.

Work on #281
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: aaf48dae-16bb-4454-aa94-bae5cd4be1ba

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/288/head:pr_288 && git checkout pr_288
cargo build

@markmandel
Copy link
Member Author

SYAC

rustc 1.51.0 (2fd73fabe 2021-03-23)
binary: rustc
commit-hash: 2fd73fabe469357a12c2c974c140f67e7cdd76d0
commit-date: 2021-03-23
host: x86_64-unknown-linux-gnu
release: 1.51.0
LLVM version: 11.0.1

@XAMPPRocky this look like what you expected?

@XAMPPRocky
Copy link
Collaborator

@markmandel Yes though it makes how clippy was producing that warning more strange, but lets see if it happens again now with this.

@markmandel markmandel merged commit c2ba704 into main Jun 3, 2021
@markmandel markmandel deleted the build/output-version branch June 3, 2021 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build-tools Development tooling. cla: yes kind/cleanup Refactoring code, fixing up documentation, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants