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

[CS2103-W14-1] Condonery #114

Open
wants to merge 605 commits into
base: master
Choose a base branch
from

Conversation

misterpuffin
Copy link

Condonery is a desktop app made for property agents primarily used for managing client contacts and condo listings. It is optimized for use via a Command Line Interface (CLI) while still having the benefits of a Graphical User Interface (GUI). If you can type fast, Condonery can get your contact management tasks done faster than traditional GUI apps.

@codecov-commenter
Copy link

codecov-commenter commented Sep 29, 2022

Codecov Report

Base: 72.15% // Head: 62.19% // Decreases project coverage by -9.96% ⚠️

Coverage data is based on head (e45dbb9) compared to base (77a32bf).
Patch coverage: 59.84% of modified lines in pull request are covered.

❗ Current head e45dbb9 differs from pull request most recent head 153835c. Consider uploading reports for the commit 153835c to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #114      +/-   ##
============================================
- Coverage     72.15%   62.19%   -9.97%     
- Complexity      399      713     +314     
============================================
  Files            70      129      +59     
  Lines          1232     2669    +1437     
  Branches        125      339     +214     
============================================
+ Hits            889     1660     +771     
- Misses          311      853     +542     
- Partials         32      156     +124     
Impacted Files Coverage Δ
src/main/java/seedu/condonery/AppParameters.java 75.00% <ø> (ø)
src/main/java/seedu/condonery/Main.java 0.00% <ø> (ø)
src/main/java/seedu/condonery/MainApp.java 0.00% <0.00%> (ø)
...main/java/seedu/condonery/commons/core/Config.java 40.90% <ø> (ø)
...java/seedu/condonery/commons/core/GuiSettings.java 69.23% <ø> (ø)
.../java/seedu/condonery/commons/core/LogsCenter.java 78.37% <ø> (ø)
...in/java/seedu/condonery/commons/core/Messages.java 0.00% <0.00%> (ø)
...ain/java/seedu/condonery/commons/core/Version.java 82.50% <ø> (ø)
...ry/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
...nery/commons/exceptions/IllegalValueException.java 100.00% <ø> (ø)
... and 123 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

yuehernkang and others added 29 commits November 1, 2022 05:23
…property-type-commands

Increase test coverage
add clearer explanations for options
add -p command's h/ prefix does not run properly
lkwlkww and others added 30 commits November 7, 2022 22:36
Fix errors in Manual Testing instructions
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.

5 participants