-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add flash formatter #27
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #27 +/- ##
=======================================
Coverage 29.85% 29.85%
=======================================
Files 27 27
Lines 1819 1819
=======================================
Hits 543 543
Misses 1276 1276 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Memory usage change @ f20f033
Click for full report table
Click for full report CSV
|
Memory usage change @ a8b52ea
Click for full report table
Click for full report CSV
|
virtual bool checkPartition() { return true; }; | ||
virtual bool formatPartition() { return true; }; |
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.
Why not pure virtual functions? Shouldn't this be an interface?
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.
Yeah maybe the name of the file is misleading, but in my idea this class is the base/default class from which derive. So in the sketch I don't have to use ifdef according to platform
Add utility for checking and formatting the Flash storage how it's needed by ArduinoIoTCloud