Skip to content

SD - #21

Open
KacSam wants to merge 5 commits into
masterfrom
SD
Open

SD #21
KacSam wants to merge 5 commits into
masterfrom
SD

Conversation

@KacSam

@KacSam KacSam commented May 22, 2024

Copy link
Copy Markdown

No description provided.

Comment thread projects/SD/src/sd.cpp
namespace sd {

void writeFile(fs::FS &fs, const char *path, const char *message) {
Serial.printf("Writing file: %s\n", path);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

dont print to serial on success scenario only on failure

Comment thread projects/SD/src/sd.cpp
}

if (file.println(message)) {
Serial.println("File written");

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same

Comment thread projects/SD/test/README
@@ -0,0 +1,11 @@

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

wtf is this readme... it can stay tho

@KajSob KajSob May 22, 2024

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 week later
brtk_rdme_po_tygodniu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants