AY1920S2-CS2103T-W17-2 / main

Notably
https://ay1920s2-cs2103t-w17-2.github.io/main/
MIT License
3 stars 4 forks source link

Remove Edit Command parser #421

Closed ljiazh3ng closed 4 years ago

ljiazh3ng commented 4 years ago

resolves #420

This PR revamp the way edit command is parsed.

Note : EditCommand currently is left empty, waiting for @HemanshuGandhi to merge his PR.

netlify[bot] commented 4 years ago

Deploy preview for notably-app ready!

Built with commit d103545a5f8d60ccfbf9cfa3095ca9f8115a1001

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

codecov[bot] commented 4 years ago

Codecov Report

Merging #421 into master will decrease coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #421      +/-   ##
============================================
- Coverage     67.11%   67.08%   -0.03%     
+ Complexity      518      515       -3     
============================================
  Files            98       97       -1     
  Lines          2025     2008      -17     
  Branches        201      199       -2     
============================================
- Hits           1359     1347      -12     
+ Misses          580      576       -4     
+ Partials         86       85       -1     
Impacted Files Coverage Δ Complexity Δ
...n/java/com/notably/logic/commands/EditCommand.java 33.33% <ø> (-6.67%) 1.00 <0.00> (ø)
...n/java/com/notably/logic/parser/NotablyParser.java 84.00% <100.00%> (ø) 10.00 <0.00> (ø)

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 d6c8ced...d103545. Read the comment docs.