daniellau88 / pe

0 stars 0 forks source link

Commit conflict in DG #7

Open daniellau88 opened 3 years ago

daniellau88 commented 3 years ago

Hi, the above conflict doesn't seem to have been resolved. image.png

nus-pe-bot commented 3 years ago

Team's Response

No response provided.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Conflict message exists in DG

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.


The DG still contains conflict messages

image.png


[original: nus-cs2103-AY2021S2/pe-interim#1885] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for pointing this out! However, this does not affect the readability of of the Developer's guide as it does not conflict with any headers or paragraphs. Having said that, we will definitely delete this in future iterations.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I agree that this bug is indeed a duplicate, but it should still be accepted as a VeryLow bug.

According to the PE guide, this should be considered a typo. image.png

And since you have also said that you would fix this in future iterations, it indirectly acknowledges that this is indeed an issue. Otherwise, there would be no reasons to fix it.

image.png