Skip to content

Added a define to format the spiffs in SPIFFS_Test.ino #1662

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Jul 25, 2018
Merged

Added a define to format the spiffs in SPIFFS_Test.ino #1662

merged 3 commits into from
Jul 25, 2018

Conversation

lbernstone
Copy link
Contributor

No description provided.

Copy link
Member

@me-no-dev me-no-dev left a 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
Copy link
Member

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)){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not changing it here?

Copy link
Contributor Author

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.

@me-no-dev me-no-dev merged commit 17065df into espressif:master Jul 25, 2018
@lbernstone lbernstone deleted the pr2 branch July 31, 2018 00:54
Curclamas pushed a commit to Curclamas/arduino-esp32 that referenced this pull request Aug 21, 2018
* Added a define to format the spiffs in SPIFFS_Test.ino

* Uncommented the define

* Matched define names
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants