Skip to content

nicer output in "commented on pagure"#464

Open
michich wants to merge 2 commits into
psss:mainfrom
michich:pagure-output
Open

nicer output in "commented on pagure"#464
michich wants to merge 2 commits into
psss:mainfrom
michich:pagure-output

Conversation

@michich
Copy link
Copy Markdown
Contributor

@michich michich commented Apr 1, 2026

Improve the output in the "commented on pagure" section. Avoid the raw HTML being printed and deduplicate listings of comments on the same pagure PR.

Before:

* Pull requests commented on pagure: 3
    * 2026-03-28 - michich commented on PR <a href="https://src.fedoraproject.org/rpms/munge/pull-request/10" title="[Open] add basic TMT tests">rpms/munge#10</a>
    * 2026-03-28 - michich commented on PR <a href="https://src.fedoraproject.org/rpms/rdma-core/pull-request/32" title="[Merged] Update rawhide to upstream release 62.0">rpms/rdma-core#32</a>
    * 2026-03-28 - michich commented on PR <a href="https://src.fedoraproject.org/rpms/rdma-core/pull-request/32" title="[Merged] Update rawhide to upstream release 62.0">rpms/rdma-core#32</a>

After:

* Pull requests commented on pagure: 2
    * rpms/munge#10 - add basic TMT tests
    * rpms/rdma-core#32 - Update rawhide to upstream release 62.0

@michich michich force-pushed the pagure-output branch 2 times, most recently from c6f4ed7 to 8f47187 Compare April 1, 2026 09:39
michich added 2 commits April 7, 2026 13:44
The Comment class was dumping HTML from description_mk directly
into the output. Parse the HTML to extract project, PR/issue
number, and title, matching the format used by other Pagure stats.

Assisted-by: Claude:claude-opus-4-6
Multiple comments on the same PR/issue produced duplicate lines
in the output. Keep only one entry per PR/issue.

Assisted-by: Claude:claude-opus-4-6
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.

1 participant