This is an automated email from the ASF dual-hosted git repository.

martijnvisser pushed a commit to branch ci_utils
in repository 
https://gitbox.apache.org/repos/asf/flink-connector-shared-utils.git


The following commit(s) were added to refs/heads/ci_utils by this push:
     new edaacb2  [FLINK-32448] Remove `main` as default branch for CI runs 
(#17)
edaacb2 is described below

commit edaacb2b7fb7d19b9aa79b144eb40bca2c447d51
Author: MartijnVisser <martijnvis...@apache.org>
AuthorDate: Tue Jun 27 20:23:29 2023 +0200

    [FLINK-32448] Remove `main` as default branch for CI runs (#17)
    
    * [FLINK-32448] Remove `main` as default branch for CI runs
    
    This results in the situation that:
    
    1. When checking out the repository that triggered a workflow, it defaults 
to the reference or SHA for that event. Otherwise, uses the default branch.
    2. When a connector branch has been specified, it uses that specific 
reference or SHA
    
    Also change the test so that the multiple-branches test:
    
    1) Uses the default branch that triggers the workflow (e.g. a PR)
    2) A specific branch
    3) Another branch that differs from 2
---
 .github/workflows/_testing.yml | 6 ++++--
 .github/workflows/ci.yml       | 1 -
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/_testing.yml b/.github/workflows/_testing.yml
index dfc8c87..ba3bfd3 100644
--- a/.github/workflows/_testing.yml
+++ b/.github/workflows/_testing.yml
@@ -42,13 +42,15 @@ jobs:
     strategy:
       matrix:
         flink_branches: [{
-          flink: 1.16.1,
-          branch: ci_utils
+          flink: 1.16.1
+#          By not specifying a branch, it should default to the branch that 
triggers this workflow
         }, {
           flink: 1.17.1,
+#          By specifying this branch, it should check out this specified branch
           branch: ci_utils
         }, {
           flink: 1.16.1,
+#          By specifying a different branch then on L49, it should check out 
this specified branch
           branch: test_project
         }]
     uses: ./.github/workflows/ci.yml
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 0eecd8f..1f9a58a 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -50,7 +50,6 @@ on:
         description: "Branch that need to be checked out"
         required: false
         type: string
-        default: main
 
 jobs:
   compile_and_test:

Reply via email to