Skip to content

Document Packit's requirements for messages emitted by dist-git - #239

Merged
betulependule merged 1 commit into
packit:mainfrom
betulependule:forgejo/dist-git-fedmsg-requirements
Oct 1, 2026
Merged

betulependule merged 1 commit into
packit:mainfrom
betulependule:forgejo/dist-git-fedmsg-requirements

Conversation

@betulependule

@betulependule betulependule commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

This PR documents what kind of information Packit requires from the messages emitted by dist-git. It's based on our existing parsers and lists what exact fields we currently parse (and descriptions of what these fields represent). It was a lot of code to go over, and I went over it multiple times (both manually and with Claude), so hopefully it all checks out.

Related to: packit/packit-service#3193

@majamassarini majamassarini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. I never thought that messages from different forgejo instances could be treated differently, good to know!

Assisted-by: Claude Sonnet 5 <noreply@anthropic.com>
@betulependule
betulependule force-pushed the forgejo/dist-git-fedmsg-requirements branch from 29d7880 to f984d51 Compare October 1, 2026 08:41
@betulependule
betulependule added this pull request to the merge queue Oct 1, 2026
Merged via the queue into packit:main with commit 69116cc Oct 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants