Skip to content

Conversation

@miabbott
Copy link
Contributor

Follow up to #1324

@github-actions github-actions bot added documentation control/skip-ci Do not run expensive CI on this job labels Jun 26, 2025
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 bootc-container-lint.md documentation to include the --no-truncate option, which prints all warnings found during the linting process. The documentation changes are straightforward and add clarity to the tool's usage.

Comment on lines 30 to 32
**\--no-truncate**\

: Print all the warnings found during the linting process
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

It would be helpful to add a period at the end of the description for consistency.

Suggested change
**\--no-truncate**\
: Print all the warnings found during the linting process
:: Print all the warnings found during the linting process.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok i added that suggestion in the latest revision

@jeckersb
Copy link
Collaborator

This stuff is semi-automatically generated via xtask, but we haven't done that in a while so it's stale. I'll go do that in a separate PR and then just reference that here.

@cgwalters
Copy link
Collaborator

The man pages are generated, it looks like in the last release or two we forgot to cargo xtask update-generated before a release.

@jeckersb
Copy link
Collaborator

#1385 will fix this along with all of the other neglected things

@jeckersb jeckersb closed this Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

control/skip-ci Do not run expensive CI on this job

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants