This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch v2-0-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit d8bb03612ce5cea9b2a452d1410240996ad1dde2 Author: Kaxil Naik <[email protected]> AuthorDate: Thu Mar 4 16:27:55 2021 +0000 Remove redundant step in CodeQL GitHub Actions step (#14600) From https://github.com/apache/airflow/runs/2031451066#step:4:9 Warning: 1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary. Please remove this step as Code Scanning recommends analyzing the merge commit for best results. I also double-check it with https://github.com/github/codeql-action (cherry picked from commit 4efd5845eb89c9e50f2668c108ed33ef0849fa0d) --- .github/workflows/codeql-analysis.yml | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index 8bdd809..c39aa68 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -66,22 +66,11 @@ jobs: - name: Checkout repository uses: actions/checkout@v2 with: - # We must fetch at least the immediate parents so that if this is - # a pull request then we can checkout the head. - fetch-depth: 2 persist-credentials: false if: | matrix.language == 'python' && needs.selective-checks.outputs.needs-python-scans == 'true' || matrix.language == 'javascript' && needs.selective-checks.outputs.needs-javascript-scans == 'true' - # If this run was triggered by a pull request event, then checkout - # the head of the pull request instead of the merge commit. - - run: git checkout HEAD^2 - if: | - github.event_name == 'pull_request' && - (matrix.language == 'python' && needs.selective-checks.outputs.needs-python-scans == 'true' || - matrix.language == 'javascript' && needs.selective-checks.outputs.needs-javascript-scans == 'true') - # Initializes the CodeQL tools for scanning. - name: Initialize CodeQL uses: github/codeql-action/init@v1
