8000 chore: build tiflash with rust cargo cache by purelind · Pull Request #3034 · PingCAP-QE/ci · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: build tiflash with rust cargo cache #3034

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

Conversation

purelind
Copy link
Collaborator

Build tiflash with rust cargo cache in nightly build.

@ti-chi-bot ti-chi-bot bot requested a review from wuhuizuo July 26, 2024 04:09
Copy link
ti-chi-bot bot commented Jul 26, 2024

I have already done a preliminary review for you, and I hope to help you do a better job.

Based on the pull request title and description, the key changes in this pull request are that Tiflash will be built using Rust cargo cache in nightly builds.

One potential problem is that it is not clear what specific version of Rust Cargo cache is being used and if there are any potential compatibility issues with the current system configuration. Another potential problem is that this change may increase the build time if Rust Cargo cache is not already installed and configured properly.

As a fix, the pull request author can provide more details about the specific version of Rust Cargo cache being used and ensure that it is compatible with the current system configuration. They can also provide instructions on how to properly install and configure Rust Cargo cache to avoid any potential build time increases. Additionally, it would be helpful to add some details about the testing that was done to ensure that this change does not introduce any new issues.

Copy link
ti-chi-bot bot commented Jul 26, 2024

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot merged commit d665704 into PingCAP-QE:main Jul 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant
0