Skip to content

Commit ea0fb85

Browse files
author
Roman Glushko
committed
#184 Changed an env variable from where PR target branch comes
1 parent 87ad45d commit ea0fb85

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

.github/workflows/gradle.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
- name: Run automated tests
2626
run: ./gradlew test -i
2727
- name: Run Code Style Check
28-
run: ./gradlew checkstyleChanged -i
28+
run: ./gradlew checkstyleCI -i
2929
- name: Run PMD Quality Check
3030
run: ./gradlew pmdMain -i
3131
- name: Run Spotbugs Quality Check
@@ -48,7 +48,7 @@ jobs:
4848
- name: Run automated tests
4949
run: ./gradlew test -i
5050
- name: Run Code Style Check
51-
run: ./gradlew checkstyleChanged -i
51+
run: ./gradlew checkstyleCI -i
5252
- name: Run PMD Quality Check
5353
run: ./gradlew pmdMain -i
5454
- name: Run Spotbugs Quality Check
@@ -71,7 +71,7 @@ jobs:
7171
- name: Run automated tests
7272
run: ./gradlew test -i
7373
- name: Run Code Style Check
74-
run: ./gradlew checkstyleChanged -i
74+
run: ./gradlew checkstyleCI -i
7575
- name: Run PMD Quality Check
7676
run: ./gradlew pmdMain -i
7777
- name: Run Spotbugs Quality Check

gradle-tasks/staticChecks.gradle

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ task checkstyle(type: Checkstyle) {
5252
}
5353

5454
// Execute Checkstyle on all modified files
55-
task checkstyleChanged(type: Checkstyle) {
55+
task checkstyleCI(type: Checkstyle) {
5656
def changedFiles = getChangedFiles()
5757
include changedFiles
5858
}
@@ -65,17 +65,17 @@ task checkstyleChanged(type: Checkstyle) {
6565
*/
6666
def getChangedFiles() {
6767
// Get the target and source branch
68-
def ghprbTargetBranch = System.getenv("ghprbTargetBranch")
69-
def ghprbSourceBranch = System.getenv("ghprbSourceBranch")
68+
def envTargetBranch = System.getenv("github.event.pull_request.head.ref")
69+
def envSourceBranch = System.getenv("ghprbSourceBranch")
7070

7171
// Compare to master if no branch specified
7272
def targetBranch
7373
if (project.hasProperty("branch")) {
7474
targetBranch = project.property("branch")
7575
} else {
76-
targetBranch = ghprbTargetBranch ? "origin/${ghprbTargetBranch}" : getParentBranch()
76+
targetBranch = envTargetBranch ? "origin/${envTargetBranch}" : getParentBranch()
7777
}
78-
def sourceBranch = ghprbSourceBranch ? "origin/${ghprbSourceBranch}" : ""
78+
def sourceBranch = envSourceBranch ? "origin/${envSourceBranch}" : ""
7979

8080
// Get list of all changed files including status
8181
def systemOutStream = new ByteArrayOutputStream()

0 commit comments

Comments
 (0)