johannagwan commented 4 years ago

Closes #450

If the frequency is the same, the ordering will follow the note's order in the hierarchy

netlify[bot] commented 4 years ago

Deploy preview for notably-app ready!

Built with commit c31a4003cc1f0a028ac2ab792f9e165fd7faf89d

https://deploy-preview-451--notably-app.netlify.com

codecov[bot] commented 4 years ago

Codecov Report

Merging #451 into master will decrease coverage by 0.03%. The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master     #451      +/-   ##
============================================
- Coverage     67.18%   67.15%   -0.04%     
- Complexity      578      581       +3     
============================================
  Files           101      101              
  Lines          2243     2259      +16     
  Branches        216      220       +4     
============================================
+ Hits           1507     1517      +10     
- Misses          660      662       +2     
- Partials         76       80       +4

Impacted Files	Coverage Δ	Complexity Δ
...c/commands/suggestion/SearchSuggestionCommand.java	`86.20% <62.50%> (-9.04%)`	`10.00 <3.00> (+3.00)`	:arrow_down:

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 6b64600...c31a400. Read the comment docs.

AY1920S2-CS2103T-W17-2 / main

Sort suggestion list based on the frequency for search command #451

Codecov Report