Skip to content

Commit

Permalink
change BASE commit for ATS
Browse files Browse the repository at this point in the history
Given how we will collect the necessary inputs for ATS
(in a periodic workflow in the master branch)
it makes more sense to use commits in the master branch as the BASE
for ATS.

The changes here change the BASE picking to those commits
(i.e. the base of a feature branch)
  • Loading branch information
giovanni-guidini committed Oct 20, 2023
1 parent 9e3a876 commit 0f5a8d3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/codecov_ats.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
# (i.e. git merge-base ${{ github.sha }}^ origin/main)
# fetch-depth: 2 - Use if the BASE_COMMIT on codecov_automated_test_selection is at known depth
# (i.e. git rev-parse ${{ github.sha }}^)
fetch-depth: 2
fetch-depth: 0
- name: Set up Python 3.10.10
uses: actions/setup-python@v4
with:
Expand Down Expand Up @@ -98,7 +98,7 @@ jobs:
- name: Codecov Automated Test Selection
id: codecov_automated_test_selection
run: |
BASE_COMMIT=$(git rev-parse ${{ github.sha }}^)
BASE_COMMIT=$(git merge-base ${{ github.sha }}^ origin/master)
echo $BASE_COMMIT
output=$(codecovcli --codecov-yml-path=codecov.yml label-analysis --dry-run --token=${CODECOV_STATIC_TOKEN} --base-sha=${BASE_COMMIT}) || true
if [ -n "${output}" ];
Expand Down

0 comments on commit 0f5a8d3

Please sign in to comment.