You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#7832 disabled running codeql and a number of other workflows in forks. This isn't terrible, but it isn't ideal. (The average repository was definitely wasting cycles, and changing the behavior saves all those cycles.)
Motivation
There are a handful of "CodeQL" "triggered" "pull requests" at the moment. It'd be handy if I could set a variable in my fork to enable using CodeQL. It would be off by default for any forks that don't have the variable defined, but it would be a lot cleaner than users having to commit to master just to run codeql in forks.
Proposal
Change this line (note, this is not the line as it exists on master, it's the line present in my fork as I had to change it in order to trigger CodeQL, the only difference being a leading #):
Summary
#7832 disabled running codeql and a number of other workflows in forks. This isn't terrible, but it isn't ideal. (The average repository was definitely wasting cycles, and changing the behavior saves all those cycles.)
Motivation
There are a handful of "CodeQL" "triggered" "pull requests" at the moment. It'd be handy if I could set a variable in my fork to enable using CodeQL. It would be off by default for any forks that don't have the variable defined, but it would be a lot cleaner than users having to commit to
master
just to run codeql in forks.Proposal
Change this line (note, this is not the line as it exists on master, it's the line present in my fork as I had to change it in order to trigger CodeQL, the only difference being a leading
#
):argo-cd/.github/workflows/codeql.yml
Line 26 in 4014cc8
To
(Eventually
enable_codeql
could be set as a var in the argoproj/argo-cd repository, and the first half of the if could be removed.)The text was updated successfully, but these errors were encountered: