-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Closed
Labels
questionFurther information is requestedFurther information is requested
Description
For an example of how being too chatty distracts from usefulness: microsoft/AttackSurfaceAnalyzer#398
Actual Behavior:
LGTM Spams my PRs with Comments for every subsequent commit if any commit previous to it in the branch fixes an LGTM issue.
Expected Behavior:
LGTM comments only on the commit that fixes the issue.
Explanation:
LGTM should comment if the new commit changes something from the previous commit (i.e. new warnings). Getting repeated information, particularly that you've fixed something, is not useful.
Metadata
Metadata
Assignees
Labels
questionFurther information is requestedFurther information is requested