search

JamesLiuZX / pe

0 stars 0 forks source link

Insufficient non-functional requirements #15

Open JamesLiuZX opened 1 year ago

JamesLiuZX commented 1 year ago

Only mentions robustness, OS compatibility and user. Can include more from here.

Screenshot 2023-04-14 at 3.33.34 PM.png

Screenshot 2023-04-14 at 3.32.28 PM.png

nus-pe-script commented 1 year ago

Team's Response

image.png

Hello! Apologies, but we will be rejecting this documentation bug based on the image attached above.

1) Looking at the bug you reported, it cannot be falling under the "unfulfilled NFRs" section since we fulfill every NFR shown in the DG, so what you must be referring to is the section "Severity of bugs related to missing requirement", however we do not find that any of the NFRs are considered as potentially damaging to omit, especially since it mentions the most important NFRs "environmental requirements" as well as "data requirements". 2) Although more definitely can be included, it is not necessary and a developers' guide is only meant to be minimal and sufficient, not all encompassing as seen in the photo attached below from the textbook, as it is meant to be read together with the source code! Looking at the developers guide as a whole, our group find that we included enough such that other developers can maintain and evolve the code!

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]