Change file path #181
Change file path #181
Conversation
ERROR:docs/ProductSurveyDraft.md:22: no newline at EOF. WARN:docs/ProductSurveyDraft.md:15: trailing whitespace. WARN:docs/ProductSurveyDraft.md:21: trailing whitespace. WARN:docs/UserGuide.md:78: trailing whitespace.
…at in developerguide
Morph AddressBook to TodoList
Update status regex and add ShowCommand
Update build.gradle
Fix markdown links in DeveloperGuide
This reverts commit f4ce861.
|
Hi @zhypaul, your pull request title is invalid. It should be in the format of Submit only one activity per pull request (unless otherwise stated in instructions) and do remember to create your branches from the commit where the Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at cs2103-pr-bot and add a link to this PR. |
…AN2017-T16-B3#181 (CS2103JAN2017-T16-B3#182) * codestyle: ensure curly brackets are separated by whitespace The Java coding standard[1] requires that curly brackets ({}) be surrounded by whitespace. However, there are several violations of this requirement in the code base. e.g. protected void raise(BaseEvent event){ -------------------------------------^ Space required before '{' Fix all of these code style violations. [1] https://oss-generic.github.io/process/codingstandards/coding-standards-java.html#methods * checkstyle: ensure curly brackets have whitespace around them There used to be methods in the code base where the curly brackets ({}) are not separated by whitespace: protected void raise(BaseEvent event){ -------------------------------------^ Space required before '{' Our checkstyle configuration failed to catch these errors because we did not tell checkstyle to check for the LCURLY, RCURLY and SLIST (statement list) tokens. Fix this in the checkstyle config file so that such code style violations will not happen again.
added command to Change storage location as user specifies.