So let's say there is a question (or answer) which has a comment I consider relevant to the question. I might have the idea to include it in the answer explaining something like
included extra information found in comments to clarify question
I thought this was best practices, I found for example this meta post supporting my thought. (The same should be true for answers.) So I have done the following:
- edited the answer to include that comment
- giving credit to the author of the comment
- fixed other issues and did improvements
- marked the said comment as "no longer needed"
I should note that I'm neither the author of the question, nor the answer I edited, nor the comment. What happened was
- flag 4 got marked as helpful and therefore accepted
- reviewer got confused as the comment I refer to no longer existed
- including the comment and the other improvements got rejected (both)
What's the best practices on that? I have some ideas:
- leave comments as they are - downside: they can be deleted, comments are generally considered less important
- not give credit and just append "Attention: this will mutate the original array." (which is to the general public, not the author btw)
- skip other changes to make it more obvious to the reviewer - downside: violates rule "fix all that needs fixing" - this might be applicable to users with >2k rep and full edit privileges tho
- don't mark the comment as deprecated, potentially remembering the suggestion threeish days later upon acception and flag it then - downside obvious: much more work
- something else? I don't see how I could let a reviewer know that a comment needs deletion if the inclusion is accepted
reverse
there it might just be possible that the comment under consideration was a reply to it.