Skip to content

app: add markdown format for the info command #39

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

Merged
merged 2 commits into from
May 27, 2025
Merged

Conversation

shettikr
Copy link
Contributor

This changeset adds a markdown format for the info command. Right now, two formats are supported - 'compact' and 'markdown'.

Usage:
$ iclg info --format markdown sample.crashlog

This changeset adds a markdown format for the info command.
Right now, two formats are supported - 'compact' and 'markdown'.

Usage:
    $ iclg info --format markdown sample.crashlog

Signed-off-by: Krishnakumar Shetti <krishnakumar.shetti@intel.com>
Signed-off-by: Surply, Pierre <pierre.surply@intel.com>
@psurply psurply merged commit c2db0a7 into intel:main May 27, 2025
5 checks passed
@psurply
Copy link
Contributor

psurply commented May 27, 2025

Looks good to me. I just changed a few things:

  • Updated the separator line to make the table compatible with the format expected by GitHub (and several other markdown parsers)
  • Moved the logic that handles the info command to the info.rs file
  • Simplified the format strings.

Thanks a lot for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants