-
Notifications
You must be signed in to change notification settings - Fork 7
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
✨ (rc): Implement BLE Service File Reception #705
✨ (rc): Implement BLE Service File Reception #705
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
|
28407d5
to
0a3b7da
Compare
Codecov Report
@@ Coverage Diff @@
## develop #705 +/- ##
===========================================
- Coverage 94.61% 94.54% -0.07%
===========================================
Files 106 106
Lines 2265 2274 +9
===========================================
+ Hits 2143 2150 +7
- Misses 122 124 +2
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
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 👍
rien à redire sauf un std::span 😂
0a3b7da
to
69baa23
Compare
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 👍
…ception - Explicit callback exist comparing it with nullptr - Remove reference to std::span
…:span<uint8_t/char>
69baa23
to
dd2c5ce
Compare
Kudos, SonarCloud Quality Gate passed! |
Need #707