Skip to content

Make NatsMsg<T>.Build Public#819

Merged
mtmk merged 7 commits into
nats-io:mainfrom
darkwatchuk:main
Apr 10, 2025
Merged

Make NatsMsg<T>.Build Public#819
mtmk merged 7 commits into
nats-io:mainfrom
darkwatchuk:main

Conversation

@darkwatchuk

Copy link
Copy Markdown
Collaborator

This PR is just to make NatsMsg.Build public , currently internal.

This allows for creation of INatsSub implementations from outside of the library.

Thanks!

@mtmk mtmk 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 @darkwatchuk

Added docs with a remark that the API is internal to give notice to application developers to be careful about using it if they are more concern about future release stability since i feel we should have the freedom to change this without much concern.

@mtmk
mtmk merged commit 8f7d81e into nats-io:main Apr 10, 2025
mtmk added a commit that referenced this pull request Apr 10, 2025
* Make NatsMsg<T>.Build Public (#819)
* Fixed build badge
@mtmk mtmk mentioned this pull request Apr 10, 2025
mtmk added a commit that referenced this pull request Apr 10, 2025
* Release 2.5.15

* Make NatsMsg<T>.Build Public (#819)
* Fixed build badge

* Update build badge
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.

2 participants