Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
same variable naming issue
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
`expectedInsurancePlansManager` instead of `expectedInsurancePlanManager`, remember to edit the javadoc too 😃
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
`expectedInsurancePlansManager` instead of `expectedInsurancePlanManager`
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
Naming would be better to follow class name. Hence `originalInsurancePlansManager` would be a better name (Note the 's')
zi-yii created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM but requires some changes to variable naming (can ctrl f to check)
zi-yii created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM but requires some changes to variable naming (can ctrl f to check)
zi-yii pushed 3 commits to master AY2425S1-CS2103-F12-1/tp
- Add tests in ParserUtilTest.java This commit adds test for parseInsurancePlan, parseClaimId and parseClaimAmount func... 7c40e18
- Change invalid claim amount to test 3 digit in cents 3706947
- Merge pull request #145 from matthewyip1511/matthew-update-parserUtilTest-CommandUtilTest Add tests in ParserUtilTest 5a4cce9