Menu

#172 Allow users to mark some violations 'reviewed' (eclipse)

Long-term
closed
5
2012-10-07
2003-08-12
No

Can the plugin for the various IDEs with a task list
(like Eclipse) allow a violation that will never be
corrected be marked "reviewed"? A reviewed violation
would not be shown in the task list or really otherwise
mentioned. This would function similar to CVS ignore.

Ignored violations should preferably be written to a
file in the eclipse environment that can be shared by
all developers. The file doesn't need to say who
reviewed it necessarily (it doesn't have a username),
but could provide a place for optional comments.
Presumably the underlying source code control system
would make it apparent as to who made the reviews.

This would be used for a type of violation that simply
cannot be gotten around. It would also make many of the
'code smell' types of evaluations more useful in the
IDE, because they can be examined, and then removed
from the task list if they're not appropriate.

Reviews could be stored not by line number, but by
method/class/attribute. If a method-level violation
exists on a method, and that method is changed, then
that violation needs to be reviewed again.

For eclipse, the reviewed violations could be presented
as a different type of PMD task, allowing the user to
filter them in or out.

Discussion

  • Tom Copeland

    Tom Copeland - 2003-08-14

    Logged In: YES
    user_id=5159

    Philippe is working on this...

    tom

     
  • Tom Copeland

    Tom Copeland - 2003-10-28

    Logged In: YES
    user_id=5159

    Fixed in latest pmd-eclipse plugin release.

    Tom

     

Log in to post a comment.

MongoDB Logo MongoDB