shahar1 commented on PR #68085: URL: https://github.com/apache/airflow/pull/68085#issuecomment-4633652090
Need your opinion - more than often we merge one PR after another, so the latest cancels the previous CodeQL runs. If we apply the `pull_request` behavior on `main` like we do in this PR, we cannot trust that the latest succeful "Check" really reflects that there are no issues (because it may have targeted different set of languages than the previous). On the other hand, as we now know - CodeQL runs are heavy, and I don't see a point to run the `CodeQL` on `Java` on non-sdk commits. Considering that anyway we have the scheduled runs - is the current PR a good compromise, should we keep the existing behavior (run all on `main`), or [on the other hand] - maybe we should even remove the runs on `main`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
