germainelee02 / pe

0 stars 0 forks source link

Able to remove the repo host even when it says in the add command that it is compulsory #16

Open germainelee02 opened 1 year ago

germainelee02 commented 1 year ago

In the add command, the h/ flag is compulsory. Yet, I can remove it later.

This inconsistency can make it confusing.

Screenshot 2023-04-14 at 3.04.44 PM.png

nus-se-bot commented 1 year ago

Team's Response

This FeatureFlaw is accepted, but at a Low severity, as while the inconsistency could be confusing, it should not affect regular usage of the product.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: This should be a medium.

This demonstrates a lack of planning in your product and its features. In the add command, repo host is compulsory, but later on, you have another command that can remove it despite it being originally compulsory.

This is quite a dire feature flaw in my opinion. So is repohost compulsory in a project, or not?