gocsaf / csaf

Tools to download or provide CSAF (Common Security Advisory Framework) documents.
https://csaf.io
40 stars 23 forks source link

Improve GH Action checks #507

Closed tschmidtb51 closed 1 year ago

tschmidtb51 commented 1 year ago

Currently, we run the go.yml just on pushs.

@s-l-teichmann Wouldn't it be useful to automatically run it on pull requests as well or was that a specific choice? If we run it on Pull Requests as well, it would make it easier for reviewers to find the obvious mistakes...

I'm happy to implement the change - I just can't remember whether this was an active decision (and if so what the reasons were).

s-l-teichmann commented 1 year ago

@tschmidtb51 Yes, we should do that and there were no active decisions not doing this. I think this a historical artifact. The amount of external PRs were small and our internal pushes are triggering the actions.