-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update cache key used in rust CI script #11641
Conversation
The comment is in `linux-build-lib` and refers to `linux-build-lib` as to some other thing.
@@ -62,8 +62,7 @@ jobs: | |||
~/.cargo/git/db/ | |||
./target/ | |||
./datafusion-cli/target/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems ok to me -- how do we know if this improves things or not 🤔
Thanks @findepi |
can we have a description, what this PR is for? |
@comphead thanks for reminder, added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it looks good for me thanks @findepi in terms of more concise text line
But I'm not sure what is the real benefit of it
technically none other than removing misleading comment & value |
Cleanup of the GitHub workflow definition.