Skip to content

Add --full-message flag to show complete commit/PR/MR bodies - #446

Merged
sandrobonazzola merged 1 commit into
psss:mainfrom
BeeGrech:add-expandedmsg
Feb 18, 2026
Merged

Add --full-message flag to show complete commit/PR/MR bodies#446
sandrobonazzola merged 1 commit into
psss:mainfrom
BeeGrech:add-expandedmsg

Conversation

@BeeGrech

@BeeGrech BeeGrech commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Adds a new CLI option that displays full content instead of just titles/subject lines:

Git commits: Shows complete commit body using %B format GitHub: Includes PR/issue body from API response
GitLab: Lazy-loads issue/MR descriptions when flag is set Multi-line content is indented with 8 spaces for readability.

@BeeGrech

BeeGrech commented Jan 9, 2026

Copy link
Copy Markdown
Contributor Author

failed test is unrelated to commit

  File "/usr/lib/python3.9/site-packages/koji/__init__.py", line 3086, in _sendOneCall
    r.raise_for_status()
  File "/usr/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 502 Server Error: Bad Gateway for url: https://koji.fedoraproject.org/kojihub

@BeeGrech

Copy link
Copy Markdown
Contributor Author
pylint...................................................................Failed
- hook id: pylint
- exit code: 8

************* Module did.plugins.git
did/plugins/git.py:42:4: R0912: Too many branches (17/12) (too-many-branches)

Would you prefer if i create a static method for the full commit message handling or could we disable too-many-branches for this function?

@sandrobonazzola

Copy link
Copy Markdown
Collaborator
pylint...................................................................Failed
- hook id: pylint
- exit code: 8

************* Module did.plugins.git
did/plugins/git.py:42:4: R0912: Too many branches (17/12) (too-many-branches)

Would you prefer if i create a static method for the full commit message handling or could we disable too-many-branches for this function?

I'm fine disabling the check for this function.

Adds a new CLI option that displays full content instead of just titles/subject lines:

Git commits: Shows complete commit body using %B format
GitHub: Includes PR/issue body from API response
GitLab: Lazy-loads issue/MR descriptions when flag is set
Multi-line content is indented with 8 spaces for readability.
@sandrobonazzola

Copy link
Copy Markdown
Collaborator

The failure on rawhide is not related to the patch. A note about the output with --format=markdown: the 8 spaces indentation is not helping much in the rendered output as it's output looks like:

* Pull requests commented on GitHub: 1
  * [psss/did#446](https://github.com/psss/did/pull/446) - Add --full-message flag to show complete commit/PR/MR bodies
        Adds a new CLI option that displays full content instead of just titles/subject lines:
        Git commits: Shows complete commit body using %B format GitHub: Includes PR/issue body from API response
        GitLab: Lazy-loads issue/MR descriptions when flag is set Multi-line content is indented with 8 spaces for readability.

which renders:

--

  • Pull requests commented on GitHub: 1
    • psss/did#446 - Add --full-message flag to show complete commit/PR/MR bodies
      Adds a new CLI option that displays full content instead of just titles/subject lines:
      Git commits: Shows complete commit body using %B format GitHub: Includes PR/issue body from API response
      GitLab: Lazy-loads issue/MR descriptions when flag is set Multi-line content is indented with 8 spaces for readability.

--

Overall, looks good enough to me anyway.

@sandrobonazzola sandrobonazzola left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sandrobonazzola
sandrobonazzola merged commit 649f4fb into psss:main Feb 18, 2026
14 of 15 checks passed
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.

2 participants