Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/store.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ function remove(id) {

// Returns the notes that match `term`.
function matches(notes, term) {
return notes.filter((note) => note.text === term);
return notes.filter((note) => note.text.includes(term));
}

function search(term) {
Expand Down
26 changes: 26 additions & 0 deletions notes.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
Changes will be found to variables and accidental characters added to one file

> Summarize what I've changed, and flag anything that looks unintended.

Read 1 file, ran 3 shell commands

Here's a summary of your uncommitted changes (working tree is ahead of the last commit; nothing is staged or pushed):

Changes

notes.js (modified)
- Line 12: usage string changed from "Usage: notes add <your note>" → "Usage: notes please add <your note>"
- Line 55: appended a line asdftrterw after the main(); call

package.json (modified)
- name: "notes-cli" → "notes-cli-1"
- version: "1.0.0" → "1.0.1"

notes.md (new, untracked)
- A two-line note: "Changes will be found to variables and accidental characters added to one file"

🚩 Flags — likely unintended
1. notes.js:55 — asdftrterw is junk. It's a stray top-level statement after main(); that does nothing but will throw a ReferenceError: asdftrterw is not defined athis is almost certainly an accidentalkeystroke. Recommend removing. 2. notes.js:12 — "Usage: notes please add" —usage hint looks accidental/incorrect; thesubcommand is add, so the message no longer matches how the command is actually invoked. Probably should revert to notes add <your note>.
3. package.json name notes-cli-1 — the -1 suffix looks like a slip rather than a deliberate rename. The version bump to 1.0.1 may be intentional, but worth confie accidental name change.

Your own notes.md even says "accidental charhat matches items #1 (and arguably #2).