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

Ppp final #558

Merged
merged 3 commits into from
Nov 14, 2023
Merged

Ppp final #558

merged 3 commits into from
Nov 14, 2023

Conversation

Gabriel4357
Copy link

No description provided.

* 'master' of https://github.com/AY2324S1-CS2103T-T13-3/tp: (35 commits)
  Change Model Implementation and Add ID
  Add search and fix PE
  Fix the tests and further code quality issue
  Add logging and assertions
  Add guard
  Add empty preamble and logs
  Add links
  Fix test cases
  Fix LoD and add logging
  Fix checkstyle
  Add assert/logger statements
  Add More Testing for missing pathways
  Cleanup uml
  Refractor ReadOnlyBook Model
  Fix ModelClassDiagram
  Clean List Command
  Update parser name
  Update DeliveryList and DeliveryView
  Rename sortFilteredDeliveryList
  Fix Sort SLAP
  ...
* 'master' of https://github.com/AY2324S1-CS2103T-T13-3/tp: (57 commits)
  Update final PPP
  Format DG
  Update Benjamin PPP
  Final line EOF
  Update DG - Refine Content
  Update UG
  Update PPP
  Change all predicates
  Fix optional
  Remove imports
  Refractor methods
  Add Author
  Add extra test cases to codebase
  Add back missing update delivery
  Clean checkstyle
  Update UG print
  Fix Checkstyle
  Add Test Coverage for non-trivial paths
  Add Test Coverage for non-trivial paths
  Cleanup transformer
  ...
@Gabriel4357 Gabriel4357 self-assigned this Nov 14, 2023
Copy link

codecov bot commented Nov 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8bfe8bc) 85.30% compared to head (b66f242) 85.30%.
Report is 12 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #558   +/-   ##
=========================================
  Coverage     85.30%   85.30%           
  Complexity     1028     1028           
=========================================
  Files           121      121           
  Lines          3164     3164           
  Branches        371      371           
=========================================
  Hits           2699     2699           
  Misses          354      354           
  Partials        111      111           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@juliusgambe juliusgambe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@juliusgambe juliusgambe merged commit ccf31de into AY2324S1-CS2103T-T13-3:master Nov 14, 2023
5 checks passed
@Gabriel4357 Gabriel4357 linked an issue Nov 14, 2023 that may be closed by this pull request
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.

Final PPP - Gabriel
2 participants