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-F09-3] ProgrammerError #90

Open
wants to merge 1,532 commits into
base: master
Choose a base branch
from

Conversation

shermannws
Copy link

ProgrammerError (P|E) is a desktop app for managing students' information, optimized for use via a Command Line Interface (CLI) while still having the benefits of a Graphical User Interface (GUI). Designed for CS2100 Lab TAs, this application is optimized to track personal particulars, grades and attendance of your students across different classes. If you are familiar with Unix commands, this is definitely for you!

@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2021

Codecov Report

Merging #90 (571e9c4) into master (ba53b8c) will decrease coverage by 10.48%.
The diff coverage is 60.76%.

❗ Current head 571e9c4 differs from pull request most recent head 3d99b60. Consider uploading reports for the commit 3d99b60 to get more accurate results
Impacted file tree graph

@@              Coverage Diff              @@
##             master      #90       +/-   ##
=============================================
- Coverage     72.15%   61.67%   -10.49%     
- Complexity      399      630      +231     
=============================================
  Files            70      104       +34     
  Lines          1232     2262     +1030     
  Branches        125      264      +139     
=============================================
+ Hits            889     1395      +506     
- Misses          311      749      +438     
- Partials         32      118       +86     
Impacted Files Coverage Δ
src/main/java/seedu/programmer/AppParameters.java 75.00% <ø> (ø)
src/main/java/seedu/programmer/Main.java 0.00% <ø> (ø)
src/main/java/seedu/programmer/MainApp.java 0.00% <0.00%> (ø)
...java/seedu/programmer/commons/core/LogsCenter.java 78.37% <ø> (ø)
...n/java/seedu/programmer/commons/core/Messages.java 0.00% <0.00%> (ø)
...ava/seedu/programmer/commons/core/index/Index.java 90.00% <ø> (ø)
...er/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
...mmer/commons/exceptions/IllegalValueException.java 100.00% <ø> (ø)
...in/java/seedu/programmer/commons/util/AppUtil.java 88.88% <ø> (ø)
.../seedu/programmer/commons/util/CollectionUtil.java 87.50% <ø> (ø)
... and 95 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...3d99b60. Read the comment docs.

okyntary pushed a commit to okyntary/tp that referenced this pull request Oct 21, 2021
…_Functionality

Added additional functionality to CCAs
HalphasX pushed a commit to HalphasX/tp that referenced this pull request Oct 26, 2021
…efactor-v1.2

Feat: Add Note Command, Add Note UI & Add Note Date Saving
shermannws and others added 27 commits November 8, 2021 09:23
Update DG for the EditLabCommand
…peiduo

Uncomment test cases for JsonProgrammerErrorStorageTest
Fix bug where duplicate students can be uploaded
Standardize spelling in UG and HelpWindow
reddevil1313 and others added 30 commits November 8, 2021 22:06
[PPP - shermannws] Fix indexing issue in Github pages
Fix for show manual testing instruction
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