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

[CS2103T-W17-1] InternSHIP #85

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

Conversation

tsm1820
Copy link

@tsm1820 tsm1820 commented Sep 15, 2021

InternSHIP helps students keep track of their internship applications. It enables students to organise their list of applications so that they do not miss any important events or deadlines pertaining to their application.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #85 (91263e7) into master (ba53b8c) will increase coverage by 11.23%.
The diff coverage is 90.79%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master      #85       +/-   ##
=============================================
+ Coverage     72.15%   83.39%   +11.23%     
- Complexity      399      757      +358     
=============================================
  Files            70       96       +26     
  Lines          1232     1933      +701     
  Branches        125      239      +114     
=============================================
+ Hits            889     1612      +723     
+ Misses          311      288       -23     
- Partials         32       33        +1     
Impacted Files Coverage Δ
src/main/java/seedu/address/Main.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...n/java/seedu/address/commons/core/GuiSettings.java 92.30% <ø> (+23.07%) ⬆️
...in/java/seedu/address/commons/core/LogsCenter.java 78.37% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
.../main/java/seedu/address/commons/util/AppUtil.java 88.88% <0.00%> (ø)
...ava/seedu/address/commons/util/CollectionUtil.java 87.50% <0.00%> (ø)
...in/java/seedu/address/commons/util/ConfigUtil.java 75.00% <0.00%> (ø)
...main/java/seedu/address/commons/util/JsonUtil.java 94.11% <0.00%> (ø)
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <ø> (ø)
... and 82 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba53b8c...91263e7. Read the comment docs.

Domszy pushed a commit to Domszy/tp that referenced this pull request Oct 19, 2021
…Increase-ViewTests

Increase code coverage for view command related classes
Nature711 and others added 28 commits November 2, 2021 23:58
…rPED

Fix and clarify stuff in the user guide
Clarify find command in user guide
…rrorMessages

* commit 'dc20a5b3f57f4267377b48eb948c336b614c3f6f':
  Clarify find command in user guide
  Clarify stuff in the user guide
This is a recommendation from the DG deliverable guide to help evaluator estimate the total project efforts.
Add an appendix named Effort in DG
…UIPaddings

* commit '0b752cce386229c4d24216b1df252764c0dac43a':
  Fix checkstyle
  Add an appendix named Effort in DG
  Small changes to some error messages
…leList

Display whole list after Accept, Reject or Complete commands
feliciaivane and others added 30 commits November 8, 2021 22:33
…ormatting

* commit 'ac504c9ef72d5e4074e4cb2b2ed46b08bab2f42e':
  Update Page Break
  Update PPP
  Fix minor mistakes in DG
  Update link
  Change uml diagram back
* master:
  Change version number to v.14
Minor fix to instruction for manual testing
Add missing line breaks
Fix minor indentation issue
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.

6 participants