Skip to content

Conversation

@Johan-Liebert1
Copy link
Collaborator

No description provided.

@bootc-bot bootc-bot bot requested a review from cgwalters August 28, 2025 09:42
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the composefs-rs dependency to a newer revision. The changes in Cargo.toml and Cargo.lock are correct. I've added a suggestion to improve the maintainability of the Cargo.toml file by centralizing the git revision for the composefs-rs dependencies using a [patch] section. This will make future updates easier.

composefs = { git = "https://github.com/containers/composefs-rs", rev = "8d1e570275621531d39b8f01681584bcc85ce01c", package = "composefs", features = ["rhel9"] }
composefs-boot = { git = "https://github.com/containers/composefs-rs", rev = "8d1e570275621531d39b8f01681584bcc85ce01c", package = "composefs-boot" }
composefs-oci = { git = "https://github.com/containers/composefs-rs", rev = "8d1e570275621531d39b8f01681584bcc85ce01c", package = "composefs-oci" }
composefs = { git = "https://github.com/containers/composefs-rs", rev = "ce28ee8a7b43290941d208c8ca820150a6504d5a", package = "composefs", features = ["rhel9"] }
Copy link
Collaborator

Choose a reason for hiding this comment

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

We could also add a

// Reviewers (including AI tools): Yes, this hash is duplicated, there's not a much better way to handle it right now.

@cgwalters cgwalters enabled auto-merge August 28, 2025 12:13
@cgwalters cgwalters merged commit 70c80c6 into bootc-dev:main Aug 28, 2025
33 of 34 checks passed
@cgwalters cgwalters added the area/composefs Issues related to composefs label Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/composefs Issues related to composefs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants