Skip to content

Make NatsJSConstants public#825

Merged
mtmk merged 2 commits into
nats-io:mainfrom
aradalvand:patch-1
Apr 14, 2025
Merged

Make NatsJSConstants public#825
mtmk merged 2 commits into
nats-io:mainfrom
aradalvand:patch-1

Conversation

@aradalvand

Copy link
Copy Markdown
Contributor

Closes #824

Comment thread src/NATS.Client.JetStream/Internal/NatsJSConstants.cs Outdated

@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 @aradalvand

@mtmk

mtmk commented Apr 14, 2025

Copy link
Copy Markdown
Member

sorry @aradalvand simple change but taking too much of your time. we need all the commits signed which means you'd need to force push. thanks!

image

@aradalvand

Copy link
Copy Markdown
Contributor Author

@mtmk No worries!
Just a simple force push? I just did, doesn't seem to have fixed the issue.

@mtmk

mtmk commented Apr 14, 2025

Copy link
Copy Markdown
Member

@mtmk No worries! Just a simple force push? I just did, doesn't seem to have fixed the issue.

I mentioned force push because you need to discard the unsigned commit and create a new signed commit with the same change. Actually it looks like your first commit was signed. it's the second one stopping the merge.

image

@aradalvand

Copy link
Copy Markdown
Contributor Author

Learned something today, lol!
Done.

@mtmk
mtmk merged commit baf7524 into nats-io:main Apr 14, 2025
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
@mtmk mtmk mentioned this pull request Apr 16, 2025
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
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.

Make NatsJSConstants public

2 participants