[GitHub] [airflow] turbaszek commented on a change in pull request #8393: Bring back CI optimisations

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

turbaszek commented on a change in pull request #8393: Bring back CI optimisations
URL: https://github.com/apache/airflow/pull/8393#discussion_r409383392
 
 

 ##########
 File path: scripts/ci/_utils.sh
 ##########
 @@ -917,34 +925,113 @@ function match_files_regexp() {
     export FILE_MATCHES
 }
 
+function get_ci_environment() {
+    if [[ ${CI:=} != "true" ]]; then
+        echo
+        echo "This is not a CI environment!. Set target branch to master and evert type to manual"
+        echo
+        export CI_EVENT_TYPE="manual"
+        export CI_TARGET_BRANCH="master"
+    else
+        if [[ ${TRAVIS:=} == "true" ]]; then
+            if [[ "${TRAVIS_PULL_REQUEST:=}" == "true" ]]; then
+                export CI_EVENT_TYPE="pull_request"
+                export CI_PR_REPO="${TRAVIS_PULL_REQUEST_SLUG}"
+                export CI_PR_BRANCH="${TRAVIS_PULL_REQUEST_BRANCH}"
+            elif [[ "${TRAVIS_EVENT_TYPE:=}" == "cron" ]]; then
+                export CI_EVENT_TYPE="schedule"
+                export CI_PR_REPO=""
+                export CI_PR_BRANCH=""
 
 Review comment:
   Can we move those variables before elifs? It seems that we are repeating it few times 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services