Skip to content

Conversation

danbugs
Copy link
Contributor

@danbugs danbugs commented Nov 18, 2024

Before 2023, Cargo recommended developers to git ignore Cargo.lock files for libraries. The official policy/recommendation for this has changed. I think we should be okay to include Cargo.lock as a way of tracking working combinations of dependencies. Plus, combining this w/ Dependabot, we should be constantly updating to the latest versions of crates regardless.

closes #37

ran cargo +nightly udeps to find then remove unused dependencies

Signed-off-by: danbugs <[email protected]>
…d Cargo.lock

ran cargo update to update the Cargo.lock

Signed-off-by: danbugs <[email protected]>
@danbugs danbugs added the area/dependencies Concerns dependencies or related components label Nov 18, 2024
jprendes
jprendes previously approved these changes Nov 18, 2024
Copy link
Contributor

@jprendes jprendes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@danbugs danbugs force-pushed the danbugs/add-cargo-lock branch from 86a7cea to a08a19c Compare November 18, 2024 20:39
@danbugs danbugs enabled auto-merge (rebase) November 18, 2024 21:01
Copy link
Contributor

@jprendes jprendes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

auto-merge was automatically disabled November 18, 2024 21:04

Base branch requires signed commits

@danbugs danbugs merged commit e5e7934 into hyperlight-dev:main Nov 18, 2024
23 checks passed
@danbugs danbugs deleted the danbugs/add-cargo-lock branch November 18, 2024 21:43
simongdavies pushed a commit to simongdavies/hyperlight that referenced this pull request Nov 18, 2024
* [dependencies] removed unused dependencies

ran cargo +nightly udeps to find then remove unused dependencies

Signed-off-by: danbugs <[email protected]>

* [dependencies] removed Cargo.lock from .gitignore and included updated Cargo.lock

ran cargo update to update the Cargo.lock

Signed-off-by: danbugs <[email protected]>

* [dependencies] bringing back uuid but as a dev-dependency

Signed-off-by: danbugs <[email protected]>

* [dependencies] re-adding rust-embed and sha256 as windows dependencies

Signed-off-by: danbugs <[email protected]>

---------

Signed-off-by: danbugs <[email protected]>
Signed-off-by: Simon Davies <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Concerns dependencies or related components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Cargo.lock
3 participants