search

nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Misleading warning message for argument mc when adding module #1037

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

The MCs for the module does satisfy the condition that 0 < mcs <= 20.

image.png

Perhaps the message should say "must be an integer where 0 < mcs <= 20".

[original: nus-cs2113-AY2223S1/pe-interim#1030] [original labels: severity.VeryLow type.FunctionalityBug]

Zhai-Yuxin commented 1 year ago

Team's Response

Error message should specify integer.

Duplicate status (if any):

--