[GitHub] [airflow] potiuk commented on pull request #27212: Enable full static checks also when only dev file change

[GitBox <gi...@apache.org>]

potiuk commented on PR #27212:
URL: https://github.com/apache/airflow/pull/27212#issuecomment-1287998381

   I updated it slightly BTW (better optimised) - only trigger full checks when .py files change in "dev".  This way we can save a quite a few build-hours when only updating dev docs for example.
   
   And for the future in case I am not around @pierrejeambrun @ashb @jedcunningham @dstandish  @ephraimbuddy  - it is super easy to test selective checks locally without invoking CI (commit-ref can be any commi-ish reference):
   
   ```
   breeze ci selective-check --commit-ref 7d014b0da5c883eac04148289d236b0f56b2a7be
   ``` 
   
   This is what it shows when I run it on the change that caused the problem:
   
   You can see:
   
   * "Running everything because env files changed message" 
   * basic-checks-only = false
   
   (Same output is printed in CI when PR runs) 
   
   
   <img width="1030" alt="Screenshot 2022-10-23 at 06 00 36" src="https://user-images.githubusercontent.com/595491/197373088-1c716550-33ac-449f-8bee-f6c686eb7b54.png">
   
   
   
   
   
   


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