-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated version to v3.11-beta #855
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #855 +/- ##
==========================================
+ Coverage 68.71% 70.40% +1.69%
==========================================
Files 39 47 +8
Lines 15207 15410 +203
Branches 3143 3161 +18
==========================================
+ Hits 10449 10849 +400
+ Misses 1700 1519 -181
+ Partials 3058 3042 -16 ☔ View full report in Codecov by Sentry. |
CMakeLists.txt
Outdated
set(NATS_VERSION_PATCH 0) | ||
set(NATS_VERSION_SUFFIX "-beta") | ||
|
||
set(NATS_VERSION_REQUIRED_NUMBER 0x030900) | ||
set(NATS_VERSION_REQUIRED_NUMBER 0x030B00) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just FYI, this number really needs to change only when there would be an "incompatibility" between versions. So until we modify/change some of the structure/functions in nats.h, there is technically no need to bump the version required number.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, sorry my mistake. I changed it back to 0A, and will update the release doc to explicitly say that it should change in the PRs that modify the API structs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.