Browse Source

Merge pull request #2165 from nextcloud/chore/noid/script-work

Improvements to analysis scripts and Drone configuration
Tim Krüger 2 years ago
parent
commit
75701a56ab
3 changed files with 12 additions and 11 deletions
  1. 5 1
      .drone.yml
  2. 7 1
      scripts/analysis/analysis-wrapper.sh
  3. 0 9
      scripts/analysis/getBranchName.sh

+ 5 - 1
.drone.yml

@@ -11,6 +11,7 @@ steps:
 trigger:
   branch:
     - master
+    - stable-*
   event:
     - pull_request
     - push
@@ -29,6 +30,7 @@ steps:
 trigger:
   branch:
     - master
+    - stable-*
   event:
     - push
     - pull_request
@@ -69,6 +71,7 @@ services:
 trigger:
   branch:
     - master
+    - stable-*
   event:
     - push
     - pull_request
@@ -91,7 +94,7 @@ steps:
       LOG_PASSWORD:
         from_secret: LOG_PASSWORD
     commands:
-      - export BRANCH=$(scripts/analysis/getBranchName.sh $GIT_USERNAME $GIT_TOKEN $DRONE_PULL_REQUEST)
+      - export BRANCH=$(git branch | grep '*' | cut -d' ' -f2)
       - scripts/analysis/analysis-wrapper.sh $GIT_USERNAME $GIT_TOKEN $BRANCH $LOG_USERNAME $LOG_PASSWORD $DRONE_BUILD_NUMBER $DRONE_PULL_REQUEST
 
   - name: notify
@@ -124,6 +127,7 @@ steps:
 trigger:
   branch:
     - master
+    - stable-*
   event:
     - push
     - pull_request

+ 7 - 1
scripts/analysis/analysis-wrapper.sh

@@ -8,9 +8,15 @@
 #6: DRONE_BUILD_NUMBER
 #7: PULL_REQUEST_NUMBER
 
-stableBranch="master"
+
 repository="talk"
 
+if [[ $3 = \stable-* ]] ; then
+  stableBranch=$3
+else
+  stableBranch="master"
+fi
+
 ruby scripts/analysis/lint-up.rb $1 $2 $3
 lintValue=$?
 

+ 0 - 9
scripts/analysis/getBranchName.sh

@@ -1,9 +0,0 @@
-#!/bin/bash
-
-# $1: username, $2: password/token, $3: pull request number
-
-if [ -z $3 ] ; then
-    echo "master";
-else
-    curl 2>/dev/null -u $1:$2 https://api.github.com/repos/nextcloud/talk-android/pulls/$3 | grep \"ref\": | grep -v master | cut -d"\"" -f4
-fi