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


##########
.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:
   Before
   ```
   [2023-08-21T22:48:22.608Z] ['info'] => Found 7 possible coverage files:
     coverage-API-sqlite.xml
     coverage-Always-sqlite.xml
     coverage-CLI-sqlite.xml
     coverage-Core-sqlite.xml
     coverage-Other-sqlite.xml
     coverage-Providers-sqlite.xml
     coverage-WWW-sqlite.xml
   ```
   
   With 6ab76c7
   ```
   [2023-08-22T09:56:59.500Z] ['info'] => Found 9 possible coverage files:
     coverage-API-sqlite-kvwxEQHj.xml
     coverage-Always-sqlite-rQoz8s85.xml
     coverage-CLI-sqlite-OPUreDDO.xml
     coverage-Core-sqlite-xwC5zEmr.xml
     coverage-Other-sqlite-p3tUHgcp.xml
     coverage-Providers-sqlite-GXp1Lb13.xml
     coverage-Providers-sqlite-NA4wWAjB.xml
     coverage-Providers-sqlite-TAi6XAr9.xml
     coverage-WWW-sqlite-8CchlrRV.xml
   ```
   
   And coverage goes back to April 2023, and become more 
[realistic](https://app.codecov.io/gh/apache/airflow/pull/33605/flags)
   
   ```
   HEAD: 84.29%
   Patch: -
   Change: 14.70%
   ```



-- 
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]

Reply via email to