search

finos / FDC3

An open standard for the financial desktop.
https://fdc3.finos.org
Other
187 stars 109 forks source link

Filter CVE scans on PRs a bit more #1213

Closed kriswest closed 1 month ago

kriswest commented 1 month ago

These are double triggering on PRs, ~not sure why - this won't entirely solve it but will reduce it~

We were triggering on both pushes and PRs, finessed the triggers as we only accept changes via PRs and we can use the synchronize trigger for when those are updated.

netlify[bot] commented 1 month ago

Deploy Preview for fdc3 canceled.

Name Link
Latest commit 5a6f34922acb3f7d154a830f98f873b0cc4d9941
Latest deploy log https://app.netlify.com/sites/fdc3/deploys/664e2cb5f218160008008ed1
kriswest commented 1 month ago

Maybe remove it on pushes? Everything is via a PR now

kriswest commented 1 month ago

@robmoffat this should be good to go - you use 'synchronize' to run whenever a PR is updated. Applied to both CVE scans and static code analysis