Skip to content

Add some consumer client docs - #185

Merged
OpsBotPrime merged 2 commits into
masterfrom
jerbaroo-consumer-docs
Oct 6, 2026
Merged

OpsBotPrime merged 2 commits into
masterfrom
jerbaroo-consumer-docs

Conversation

@jerbaroo

Copy link
Copy Markdown
Contributor

Wrote some comments as I was trying to understand this code. Hopefully it helps a future reader.

@ReinierMaas ReinierMaas 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! Thanks for the additional documentation. I only require one small revert.

Comment thread opsqueue/src/consumer/client.rs
@jerbaroo

jerbaroo commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

@OpsBotPrime merge

jerbaroo and others added 2 commits October 6, 2026 09:18
Approved-by: jerbaroo
Priority: Normal
Auto-deploy: false
@OpsBotPrime

Copy link
Copy Markdown
Contributor

Rebased as 31a7488, waiting for CI …

@OpsBotPrime

Copy link
Copy Markdown
Contributor

CI job 🟡 started.

@OpsBotPrime
OpsBotPrime force-pushed the jerbaroo-consumer-docs branch from 1441a5a to 31a7488 Compare October 6, 2026 07:25
@OpsBotPrime
OpsBotPrime merged commit 31a7488 into master Oct 6, 2026
6 of 7 checks passed
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.

3 participants