[w5.7][T12-2] Li Xinze

[lixinze777]

The find function is now case-insensitive

[nus-se-pr-bot]

Hi @lixinze777, your pull request title is invalid.

For PR sent to satisfy a learning outcome, the PR name should be in the format of [Learning Outcome ID][Team ID] Your Name, where [Learning Outcome ID] has no dashes or spaces (e.g. [W3.1a]) and [Team ID] has one dash only and no spaces (e.g. [W14-2] means Wednesday 2pm (14 hrs), Team 2).
E.g. If you are in tutorial W09 (i.e. Wednesday 9am), team 1, and is submitting a PR for LO W2.2b, the PR title would be [W2.2b][W09-1] James Yong. Note that your tutorial IDs are different from those shown in CORS/IVLE.

For team PR, the PR name should be in the format of [Team ID] Product Name (e.g. [T09-2] Contact List Pro).

Please follow the above format strictly and edit your title for reprocessing.

Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at nus-se-pr-bot and add a link to this PR.

[eugenepeh]

Interesting, however since you've made the modification to make the find command case insensitive, perhaps you should update the documentation and test cases to ensure its working properly as intended.

Good job on completing the activity.
Some comments added. Please close the PR after reading the comments.

[eugenepeh]

remember to disable intellij auto import wildcard
Why is wild card import bad?

[eugenepeh]

as mentioned during the lecture, such tags with no further explanation or variable name is self explanatory, can exclude them from the javadocs.