-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Added a define to format the spiffs in SPIFFS_Test.ino #1662
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
Conversation
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.
Please see the comment I left :)
// Format SPIFFS the first time you run a test | ||
// or else use the SPIFFS plugin to create a partition | ||
// https://github.com/me-no-dev/arduino-esp32fs-plugin | ||
//#define FORMAT_SPIFFS 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.
uncomment this define :) tests are failing. Maybe rename to FORMAT_SPIFFS_IF_FAILED
to be more clear when it will be formatted.
@@ -151,7 +156,7 @@ void testFileIO(fs::FS &fs, const char * path){ | |||
|
|||
void setup(){ | |||
Serial.begin(115200); | |||
if(!SPIFFS.begin()){ | |||
if(!SPIFFS.begin(FORMAT_SPIFFS)){ |
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.
not changing it here?
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.
I need AI. Mine is broken.
* Added a define to format the spiffs in SPIFFS_Test.ino * Uncommented the define * Matched define names
No description provided.