Skip to content
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

Add remark feature #63

Merged
merged 23 commits into from
Apr 2, 2024

Fix failing test cases

ff45099
Select commit
Loading
Failed to load commit list.
Merged

Add remark feature #63

Fix failing test cases
ff45099
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 31, 2024 in 0s

78.31% of diff hit (target 74.82%)

View this Pull Request on Codecov

78.31% of diff hit (target 74.82%)

Annotations

Check warning on line 81 in src/main/java/seedu/address/logic/parser/AddressBookParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/AddressBookParser.java#L81

Added line #L81 was not covered by tests

Check warning on line 18 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L18

Added line #L18 was not covered by tests

Check warning on line 28 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L27-L28

Added lines #L27 - L28 were not covered by tests

Check warning on line 35 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L33-L35

Added lines #L33 - L35 were not covered by tests

Check warning on line 37 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L37

Added line #L37 was not covered by tests

Check warning on line 39 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L39

Added line #L39 was not covered by tests

Check warning on line 41 in src/main/java/seedu/address/logic/parser/RemarkCommandParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/logic/parser/RemarkCommandParser.java#L41

Added line #L41 was not covered by tests

Check warning on line 115 in src/main/java/seedu/address/model/person/Person.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/model/person/Person.java#L115

Added line #L115 was not covered by tests

Check warning on line 31 in src/main/java/seedu/address/model/person/Remark.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/model/person/Remark.java#L31

Added line #L31 was not covered by tests

Check warning on line 40 in src/main/java/seedu/address/model/person/Remark.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/model/person/Remark.java#L40

Added line #L40 was not covered by tests

Check warning on line 63 in src/main/java/seedu/address/ui/PersonCard.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/seedu/address/ui/PersonCard.java#L63

Added line #L63 was not covered by tests