Skip to content

Conversation

@Johan-Liebert1
Copy link
Collaborator

Rename the function kargs_from_composefs_filesystem to
compute_new_kargs as it now has nothing to do with composefs

Fix a few typos across the project. Also update some comments/error
messages

Rename the function `kargs_from_composefs_filesystem` to
`compute_new_kargs` as it now has nothing to do with composefs

Signed-off-by: Pragyan Poudyal <[email protected]>
@github-actions github-actions bot added area/install Issues related to `bootc install` area/ostree Issues related to ostree labels Nov 26, 2025
@bootc-bot bootc-bot bot requested a review from jmarrero November 26, 2025 10:04
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 delivers on its promise of minor fixes, primarily through a function rename for better clarity (kargs_from_composefs_filesystem to compute_new_kargs) and a host of typo corrections across documentation, comments, and log messages. The function rename is accompanied by a much-improved docstring that clearly explains its purpose and parameters. The various typo fixes enhance the overall readability and professionalism of the codebase. All changes are correct and contribute positively to code quality. The pull request is well-executed and ready to be merged.

Fix a few typos across the project. Also update some comments/error
messages

Signed-off-by: Pragyan Poudyal <[email protected]>
@cgwalters cgwalters merged commit 2c34df6 into bootc-dev:main Dec 1, 2025
44 of 47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/install Issues related to `bootc install` area/ostree Issues related to ostree

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants