Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

Documentation review #224

Merged
merged 18 commits into from
Nov 13, 2023
Merged

Documentation review #224

merged 18 commits into from
Nov 13, 2023

Conversation

bwangpj
Copy link

@bwangpj bwangpj commented Nov 13, 2023

Documentation review

@bwangpj bwangpj added this to the v1.4 milestone Nov 13, 2023
Copy link

codecov bot commented Nov 13, 2023

Codecov Report

Merging #224 (fddc33f) into master (b7f2277) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head fddc33f differs from pull request most recent head 90dfa03. Consider uploading reports for the commit 90dfa03 to get more accurate results

@@            Coverage Diff            @@
##             master     #224   +/-   ##
=========================================
  Coverage     74.52%   74.52%           
  Complexity      459      459           
=========================================
  Files            76       76           
  Lines          1484     1484           
  Branches        138      138           
=========================================
  Hits           1106     1106           
  Misses          351      351           
  Partials         27       27           

docs/UserGuide.md Outdated Show resolved Hide resolved
@Cloud7050
Copy link

Good find on the flickr link.

This was referenced Nov 13, 2023
@Cloud7050 Cloud7050 self-requested a review November 13, 2023 13:47
docs/DeveloperGuide.md Outdated Show resolved Hide resolved
docs/diagrams/ModelClassDiagram.puml Outdated Show resolved Hide resolved
@Cloud7050 Cloud7050 mentioned this pull request Nov 13, 2023
@Cloud7050 Cloud7050 self-requested a review November 13, 2023 13:56
@Cloud7050 Cloud7050 merged commit e4f9fab into AY2324S1-CS2103-W14-3:master Nov 13, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants