I am reviewing a big C# code base and thought on how to mark it as reviewed when it is, in order to not review it twice.
Currently I have a ReviewedAttribute
which I use to decorate members that have been reviewed:
public class ReviewedAttribute : Attribute
{
}
That approach however, misses an important aspect:
- How to find code that hasn't been reviewed yet ?
Easy you'll tell me, a code block that doesn't have it hasn't been reviewed ! The problem with this approach is that you'd have to open each file to see if the attribute is in it ...
I could also add a NotReviewed
attribute on each place that hasn't been reviewed yet but it's an extremely tedious approach as well ...
Basically I'm looking for an approach like XML documentation does when it's enabled : a warning issued for each place that hasn't been documented.
Since if I haven't found such a way, I'm starting to think this is not the best approach.
Question:
Is there an effective strategy to mark blocks of code as reviewed ?