addressbook-level2 icon indicating copy to clipboard operation
addressbook-level2 copied to clipboard

Remove unnecessary semi-colon #347

Open dezhanglee opened this issue 5 years ago • 4 comments

Fixes #347

This PR removes a redundant ; from the Command class.

dezhanglee avatar Dec 15 '19 06:12 dezhanglee

Click here to submit a new iteration when this PR is ready for review.

See this repository's contribution guide for more information.

canihasreview[bot] avatar Dec 15 '19 06:12 canihasreview[bot]

v1

@dezhanglee submitted v1 for review.

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level2.git refs/pr/512/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

canihasreview[bot] avatar Dec 15 '19 06:12 canihasreview[bot]

@dezhanglee You might want to check out the contributor's guide section on commit messages. The subject line of the commit message should be written in the correct tense. The changes made in the commit should be obvious to the reader even if they are not on github.

You can look through previously merged PRs for examples.

j-lum avatar Dec 16 '19 15:12 j-lum

hi @j-lum , is my commit message ok now?

dezhanglee avatar Dec 17 '19 14:12 dezhanglee