openshift-pipelines / pipelines-as-code

Pipelines-as-Code for Tekton
https://pipelinesascode.com
Apache License 2.0
124 stars 81 forks source link

Only act on events for newly created comments #1646

Closed chmouel closed 3 months ago

chmouel commented 3 months ago

When parsing issues comments, only do this on newly created comment not for the other events.

This was an issue because when deleting a comment it would fire the on-comment event and run a pipelinerun which is probably not what the user wanted.

Fixes #1645

Signed-off-by: Chmouel Boudjnah chmouel@redhat.com

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.12%. Comparing base (3274759) to head (508a638).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1646 +/- ## ======================================= Coverage 64.11% 64.12% ======================================= Files 140 140 Lines 10912 10915 +3 ======================================= + Hits 6996 6999 +3 Misses 3382 3382 Partials 534 534 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.