Skip to content
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

chore(CI): validate Cargo lock file #2309

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

chore(CI): validate Cargo lock file #2309

wants to merge 2 commits into from

Conversation

laruh
Copy link
Member

@laruh laruh commented Dec 27, 2024

No description provided.

mariocynicys
mariocynicys previously approved these changes Dec 27, 2024
Copy link
Collaborator

@mariocynicys mariocynicys left a comment

Choose a reason for hiding this comment

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

Thanks!
LGTM

borngraced
borngraced previously approved these changes Dec 27, 2024
Copy link
Member

Choose a reason for hiding this comment

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

I would create separate pipeline for checking Cargo.lock

Comment on lines 32 to 33
- name: Backup Cargo.lock
run: cp Cargo.lock Cargo.lock.bak
Copy link
Member

Choose a reason for hiding this comment

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

We could simply run cargo update -w --locked which will fail if Cargo.lock isn't up-to-date.

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool, does the job!
https://github.com/KomodoPlatform/komodo-defi-framework/actions/runs/12526268691/job/34938647572

      Adding web-time v1.1.0
error: the lock file /home/runner/work/komodo-defi-framework/komodo-defi-framework/Cargo.lock needs to be updated but --locked was passed to prevent this

commit 4d83f94

@laruh laruh dismissed stale reviews from borngraced and mariocynicys via 3fb42e3 December 28, 2024 11:37
@laruh laruh force-pushed the ci-check-cargo-lock branch from 3fb42e3 to 4d83f94 Compare December 28, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants