-
Notifications
You must be signed in to change notification settings - Fork 8
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
✨ (ble): Add os version in advertising data #1186
✨ (ble): Add os version in advertising data #1186
Conversation
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
1dbbd35
to
2aef9a3
Compare
Codecov Report
@@ Coverage Diff @@
## develop #1186 +/- ##
===========================================
+ Coverage 96.10% 96.11% +0.01%
===========================================
Files 146 146
Lines 3544 3554 +10
===========================================
+ Hits 3406 3416 +10
Misses 138 138
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Visible with nRF Connect |
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 👍
just a very small suggestion.
will this work with the current or next version of the app? the value will be visible without connection?
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
2aef9a3
to
9e31f72
Compare
9e31f72
to
ec8efb6
Compare
rebased |
Kudos, SonarCloud Quality Gate passed! |
To be visible without connection, it will avoid to connect to a robot and check its version and go back if it is already updated