Skip to content
This repository was archived by the owner on Apr 2, 2026. It is now read-only.

Update lockfile and assert up-to-date#904

Closed
tisonkun wants to merge 2 commits into
zesterer:mainfrom
tisonkun:deplock
Closed

Update lockfile and assert up-to-date#904
tisonkun wants to merge 2 commits into
zesterer:mainfrom
tisonkun:deplock

Conversation

@tisonkun

@tisonkun tisonkun commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: tison <wander4096@gmail.com>
Comment thread Cargo.lock
Comment on lines -181 to +191
version = "0.11.2"
version = "0.12.0"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the outdated lock info even if not update.

@zesterer

zesterer commented Nov 7, 2025

Copy link
Copy Markdown
Owner

Hmm, I think it would simply be better to use --frozen in the existing CI steps to ensure that the lockfile was updated for each dependency change, no?

@tisonkun

tisonkun commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

Reasonable. Let me add it to cargo clippy.

Signed-off-by: tison <wander4096@gmail.com>
@tisonkun

tisonkun commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

Updated at ae62501

@tisonkun

Copy link
Copy Markdown
Contributor Author

reminder @zesterer now the patch should be ready for another review :D

@zesterer

Copy link
Copy Markdown
Owner

Oh, to clarify: I meant all of the steps. But the lockfile is now out of date (due to upstream changes) so I might close this PR and make the change separately, unless you were interested in updating it?

@tisonkun tisonkun closed this Nov 22, 2025
@tisonkun

Copy link
Copy Markdown
Contributor Author

As long as it would be fixed, I'm leaving the task now.

If I find further issues, I'd file a new PR :D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants