Monerh A

[Miuroro]

No description provided.

[mvcatsifma]

Taking this review @Miuroro

[mvcatsifma]

@Miuroro

Solid work — the implementation is largely correct and the output is almost identical to the reference.

A few points to improve:

Output verification
The README asks you to compare actual vs expected output. You’re one line off: learned returns 1 instead of true because the raw SQLite value isn’t converted. A quick diff would have caught this immediately.

Commit messages
"finished implementing files for the flashcard app task-2" describes your process, not the change. Prefer messages like:
Implement SQLite storage layer in database.js

Tests
There are no tests. Without them, correctness depends on manually running the app and checking output. Even a few simple assertions would catch issues like this and prevent regressions.

Error handling
The try/catch blocks in database.js catch real errors and rethrow them as "Not implemented", which hides the actual failure. Let errors propagate so they remain debuggable.

cc @stasel

[Miuroro]

thank you for the feedback ill check my code again and make the necessary changes

[github-actions]

📝 HackYourFuture auto grade

Assignment Score: 0 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 0
🧪 The auto grade is experimental and still being improved

Test Details