damithc / foo

0 stars 0 forks source link

[DG] DuplicateTaskException #47

Open damithc opened 4 years ago

damithc commented 4 years ago

actual destination: nus-cs2113-ay1920s2/pe-moderation

Issue Description

:information_source: Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


DuplicateTaskException was not thrown when Lab 4 already exists as a task and lab 4 is added.


[original: benchan911/pe#10]


Team's Response

The names of the directories are case-sensitive. This was intended as it simulates how file names are also case-sensitive in some file systems, and LINUX also has case-sensitve directory names. Thus, names of directories that are "same" and only differ in cases will be considered as different names. The only exception is module code, since they are unique regardless of cases, and are not a user-made name. This can be shown in the User Guide:

image.png

However, we are willing to accept the bug since we may have failed to adequately convey this difference. Thank you for pointing this bug out.

To Check [1 Items]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: This is not a cosmetic issue, this is an issue about how the user interacts with Nuke. Therefore, it should be Low instead of VeryLow. However, if this problem is consistent throughout Nuke, then it is more reasonable to say its a very low documentation error which was not stated clearly.