CS2103-AY1819S2-W10-1 / main

📖☕ README integrated bookmark manager and feed reader application
https://cs2103-ay1819s2-w10-1.github.io/main/
MIT License
3 stars 5 forks source link

Promote subscribe and add to all contexts #182

Closed qjqqyy closed 5 years ago

qjqqyy commented 5 years ago

Summary of changes

  1. promotes subscribe to all contexts (closes #153)
  2. rename feed context add to addindex (alias addi) prevents collision with 3
  3. promotes add to all contexts for consistency with 1
  4. select command has been lying in multiple (EntryBookSearchParser and EntryBookFeedsParser) parsers on top of being in the base EntryBookParser, so we standardise it by removing the SelectCommand reference in EntryBookParser. rationale is that select will behave differently in future versions in feeds context

while here improve some Parser tests, while here also add help message in bing command since it was missing.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 832


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/seedu/address/logic/parser/AddIndexCommandParser.java 2 3 66.67%
src/main/java/seedu/address/logic/parser/EntryBookArchivesParser.java 0 1 0.0%
src/main/java/seedu/address/logic/parser/EntryBookParser.java 1 2 50.0%
<!-- Total: 8 11 72.73% -->
Totals Coverage Status
Change from base Build 830: 2.2%
Covered Lines: 2143
Relevant Lines: 2520

💛 - Coveralls