Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Ship the Cargo.lock file in the source distribution #7

Merged
merged 2 commits into from
Jan 7, 2022
Merged

Ship the Cargo.lock file in the source distribution #7

merged 2 commits into from
Jan 7, 2022

Conversation

cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Jan 5, 2022

@cpcloud cpcloud mentioned this pull request Jan 5, 2022
Closed
@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 5, 2022

cc @jimexist

@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 7, 2022

This is failing because the cache key used in CI invalid. I'll fix that here.

@jimexist
Copy link
Contributor

jimexist commented Jan 7, 2022

thanks @cpcloud - do you want to include the lockfile itself into git?

@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 7, 2022

Yeah, that's the plan. I'll finish it up tomorrow.

@cpcloud cpcloud merged commit 6635379 into datafusion-contrib:main Jan 7, 2022
@cpcloud cpcloud deleted the test-cargo-lock-freshness branch January 7, 2022 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@houqp houqp houqp approved these changes

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
@cpcloud @jimexist @houqp