Skip to content

Feat[bmq]: bmqa::Message::totalSize - #1247

Merged
dorjesinpo merged 2 commits into
mainfrom
dev/get-properties-size
Apr 9, 2026
Merged

Feat[bmq]: bmqa::Message::totalSize#1247
dorjesinpo merged 2 commits into
mainfrom
dev/get-properties-size

Conversation

@dorjesinpo

@dorjesinpo dorjesinpo commented Apr 1, 2026

Copy link
Copy Markdown
Collaborator

instead of adding bmqa::Message::messagePropertiesAndPaddingSize, adding bmqa::Message::totalSize

@dorjesinpo dorjesinpo added the enhancement New feature or request label Apr 1, 2026
@dorjesinpo
dorjesinpo requested a review from a team as a code owner April 1, 2026 22:33
@dorjesinpo
dorjesinpo force-pushed the dev/get-properties-size branch from 2c712c5 to 74b6be2 Compare April 1, 2026 23:51
@dorjesinpo dorjesinpo changed the title bmqa::Message::messagePropertiesAndPaddingSize Feat[bmq]: bmqa::Message::messagePropertiesAndPaddingSize Apr 2, 2026
Signed-off-by: dorjesinpo <129227380+dorjesinpo@users.noreply.github.com>
Signed-off-by: dorjesinpo <129227380+dorjesinpo@users.noreply.github.com>
@dorjesinpo
dorjesinpo force-pushed the dev/get-properties-size branch from 74b6be2 to 2a0f63f Compare April 9, 2026 17:31
@dorjesinpo dorjesinpo changed the title Feat[bmq]: bmqa::Message::messagePropertiesAndPaddingSize Feat[bmq]: bmqa::Message::totalSize Apr 9, 2026

@pniedzielski pniedzielski 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.

Easiest PR to review ever, happy to see the tests too.

Safe to squash-merge

@pniedzielski

Copy link
Copy Markdown
Collaborator

I'll let the CI go through before merging

@dorjesinpo
dorjesinpo merged commit 04c3766 into main Apr 9, 2026
50 of 51 checks passed
@dorjesinpo
dorjesinpo deleted the dev/get-properties-size branch April 9, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants