Skip to content

Update PPP#42

Merged
dlathyun merged 7 commits intoAY2324S1-CS2103T-W10-2:masterfrom
dlathyun:dlathyun/update-PPP
Oct 4, 2023
Merged

Update PPP#42
dlathyun merged 7 commits intoAY2324S1-CS2103T-W10-2:masterfrom
dlathyun:dlathyun/update-PPP

Conversation

@dlathyun
Copy link
Copy Markdown

@dlathyun dlathyun commented Oct 4, 2023

Update dlathyun's PPP so that the opening sentences are standardised across our team.

@dlathyun dlathyun requested a review from alyssaongyx October 4, 2023 05:52
@dlathyun dlathyun self-assigned this Oct 4, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 4, 2023

Codecov Report

Merging #42 (1a8ef49) into master (8f6a613) will not change coverage.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage     75.26%   75.26%           
  Complexity      419      419           
=========================================
  Files            71       71           
  Lines          1338     1338           
  Branches        126      126           
=========================================
  Hits           1007     1007           
  Misses          301      301           
  Partials         30       30           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dlathyun dlathyun merged commit afb5588 into AY2324S1-CS2103T-W10-2:master Oct 4, 2023
@aliciamichellew
Copy link
Copy Markdown

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants