Skip to content

Add ConsumerProducer application#387

Merged
raphala merged 110 commits intomasterfrom
feature/add-consumerproducer-app
Dec 4, 2025
Merged

Add ConsumerProducer application#387
raphala merged 110 commits intomasterfrom
feature/add-consumerproducer-app

Conversation

@raphala
Copy link
Contributor

@raphala raphala commented Nov 12, 2025

No description provided.

Copy link
Member

@jkbe jkbe left a comment

Choose a reason for hiding this comment

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

Some naming issues are similar to #385, e.g. there are a couple of results in files that match *consumerproducer* for /application.?id/ (should likely be group ID) and also streams (should often be consumer producer)

@raphala raphala requested a review from jkbe November 28, 2025 13:45
Copy link
Member

@jkbe jkbe left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

Base automatically changed from feature/add-consumer-app to master December 1, 2025 16:24
@raphala raphala requested a review from jkbe December 1, 2025 16:59
jkbe
jkbe previously approved these changes Dec 1, 2025
jkbe
jkbe previously approved these changes Dec 1, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2025

@raphala raphala merged commit 7e28af0 into master Dec 4, 2025
24 checks passed
@raphala raphala deleted the feature/add-consumerproducer-app branch December 4, 2025 09:15
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