[GitHub] [airflow] Taragolis commented on a diff in pull request #33605: Add random suffix to coverage reports

["Taragolis (via GitHub)" <gi...@apache.org>]

Taragolis commented on code in PR #33605:
URL: https://github.com/apache/airflow/pull/33605#discussion_r1301530578


##########
.github/workflows/ci.yml:
##########
@@ -46,7 +46,7 @@ env:
         secrets.CONSTRAINTS_GITHUB_REPOSITORY || 'apache/airflow' }}
   # In builds from forks, this token is read-only. For scheduler/direct push it is WRITE one
   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-  ENABLE_TEST_COVERAGE: "${{ github.event_name == 'push' }}"
+  ENABLE_TEST_COVERAGE: "true"

Review Comment:
   It just workaround to check it into the branch is it work or not, until merge to the master. I will revert this commit https://github.com/apache/airflow/pull/33605/commits/d562e629612ddd9252799fe014c8c5b5d48b05cb when finish



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org