Skip to content

Commit 24d80a5

Browse files
committed
build fixes
1 parent 60f8a0a commit 24d80a5

5 files changed

Lines changed: 35 additions & 303 deletions

File tree

.github/workflows/build.yml

Lines changed: 17 additions & 89 deletions
Original file line numberDiff line numberDiff line change
@@ -6,23 +6,25 @@ on:
66
pull_request:
77
branches: [ master ]
88

9+
# Add permissions for test reporting
10+
permissions:
11+
contents: read
12+
checks: write
13+
pull-requests: write
14+
915
jobs:
1016
test:
11-
name: Test on Java ${{ matrix.java-version }}
17+
name: Test on Java 21
1218
runs-on: ubuntu-latest
1319

14-
strategy:
15-
matrix:
16-
java-version: [17, 21]
17-
1820
steps:
1921
- name: Checkout repository
2022
uses: actions/checkout@v4
2123

22-
- name: Set up JDK ${{ matrix.java-version }}
24+
- name: Set up JDK 21
2325
uses: actions/setup-java@v4
2426
with:
25-
java-version: ${{ matrix.java-version }}
27+
java-version: '21'
2628
distribution: 'temurin'
2729

2830
- name: Setup Gradle
@@ -33,94 +35,20 @@ jobs:
3335
- name: Run tests
3436
run: ./gradlew test --no-daemon
3537

36-
- name: Generate test report
37-
uses: dorny/test-reporter@v1
38-
if: success() || failure()
38+
- name: Publish Test Results
39+
uses: EnricoMi/publish-unit-test-result-action@v2
40+
if: always()
3941
with:
40-
name: Test Results (Java ${{ matrix.java-version }})
41-
path: build/test-results/test/*.xml
42-
reporter: java-junit
42+
files: |
43+
build/test-results/test/*.xml
44+
check_name: "Test Results (Java 21)"
45+
comment_title: "Test Results (Java 21)"
4346

4447
- name: Upload test results
4548
uses: actions/upload-artifact@v4
4649
if: always()
4750
with:
48-
name: test-results-java-${{ matrix.java-version }}
51+
name: test-results-java-21
4952
path: |
5053
build/reports/tests/
5154
build/test-results/
52-
53-
build:
54-
name: Build Application
55-
runs-on: ubuntu-latest
56-
needs: test
57-
58-
steps:
59-
- name: Checkout repository
60-
uses: actions/checkout@v4
61-
62-
- name: Set up JDK 17
63-
uses: actions/setup-java@v4
64-
with:
65-
java-version: '17'
66-
distribution: 'temurin'
67-
68-
- name: Setup Gradle
69-
uses: gradle/gradle-build-action@v2
70-
with:
71-
gradle-version: wrapper
72-
73-
- name: Build application
74-
run: ./gradlew build --no-daemon
75-
76-
- name: Upload build artifacts
77-
uses: actions/upload-artifact@v4
78-
with:
79-
name: build-artifacts
80-
path: |
81-
build/libs/*.jar
82-
build/distributions/*
83-
84-
# Only run on master branch pushes
85-
release:
86-
name: Create Release
87-
runs-on: ubuntu-latest
88-
needs: [test, build]
89-
if: github.ref == 'refs/heads/master' && github.event_name == 'push'
90-
91-
steps:
92-
- name: Checkout repository
93-
uses: actions/checkout@v4
94-
95-
- name: Set up JDK 17
96-
uses: actions/setup-java@v4
97-
with:
98-
java-version: '17'
99-
distribution: 'temurin'
100-
101-
- name: Setup Gradle
102-
uses: gradle/gradle-build-action@v2
103-
with:
104-
gradle-version: wrapper
105-
106-
- name: Build release
107-
run: ./gradlew build --no-daemon
108-
109-
- name: Get version
110-
id: version
111-
run: echo "VERSION=$(date +'%Y.%m.%d')-${{ github.run_number }}" >> $GITHUB_OUTPUT
112-
113-
- name: Create Release
114-
uses: actions/create-release@v1
115-
env:
116-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
117-
with:
118-
tag_name: v${{ steps.version.outputs.VERSION }}
119-
release_name: Release v${{ steps.version.outputs.VERSION }}
120-
body: |
121-
Automated release from main branch
122-
123-
Changes in this release:
124-
${{ github.event.head_commit.message }}
125-
draft: false
126-
prerelease: false

.github/workflows/ci.yml

Lines changed: 0 additions & 147 deletions
This file was deleted.

.github/workflows/static-analysis.yml

Lines changed: 2 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -15,33 +15,23 @@ jobs:
1515
- name: Checkout repository
1616
uses: actions/checkout@v4
1717

18-
- name: Set up JDK 17
18+
- name: Set up JDK 21
1919
uses: actions/setup-java@v4
2020
with:
21-
java-version: '17'
21+
java-version: '21'
2222
distribution: 'temurin'
2323

2424
- name: Setup Gradle
2525
uses: gradle/gradle-build-action@v2
2626
with:
2727
gradle-version: wrapper
2828

29-
- name: Run SpotBugs analysis
30-
run: ./gradlew spotbugsMain --no-daemon
31-
3229
- name: Run PMD analysis
3330
run: ./gradlew pmdMain --no-daemon
3431

3532
- name: Run Checkstyle
3633
run: ./gradlew checkstyleMain --no-daemon
3734

38-
- name: Upload SpotBugs results
39-
uses: actions/upload-artifact@v4
40-
if: always()
41-
with:
42-
name: spotbugs-results
43-
path: build/reports/spotbugs/
44-
4535
- name: Upload PMD results
4636
uses: actions/upload-artifact@v4
4737
if: always()
@@ -55,32 +45,3 @@ jobs:
5545
with:
5646
name: checkstyle-results
5747
path: build/reports/checkstyle/
58-
59-
security-scan:
60-
name: Security Vulnerability Scan
61-
runs-on: ubuntu-latest
62-
63-
steps:
64-
- name: Checkout repository
65-
uses: actions/checkout@v4
66-
67-
- name: Set up JDK 17
68-
uses: actions/setup-java@v4
69-
with:
70-
java-version: '17'
71-
distribution: 'temurin'
72-
73-
- name: Setup Gradle
74-
uses: gradle/gradle-build-action@v2
75-
with:
76-
gradle-version: wrapper
77-
78-
- name: Run OWASP dependency check
79-
run: ./gradlew dependencyCheckAnalyze --no-daemon
80-
81-
- name: Upload security scan results
82-
uses: actions/upload-artifact@v4
83-
if: always()
84-
with:
85-
name: security-scan-results
86-
path: build/reports/dependency-check-report.*

README.md

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ A Spring Boot application demonstrating advanced Temporal workflow patterns, ext
88
## 🚀 Quick Start
99

1010
### Prerequisites
11-
- Java 17 or later
11+
- Java 21 or later
1212
- [Temporal Server](https://learn.temporal.io/getting_started/java/dev_environment/) running locally
1313

1414
### Build and Run
@@ -56,17 +56,13 @@ java -jar build/libs/temporal-example-1.0.0.jar
5656
# Generate code coverage report
5757
./gradlew jacocoTestReport
5858

59-
# Run static analysis tools
60-
./gradlew staticAnalysis
61-
62-
# Run security vulnerability scan
63-
./gradlew dependencyCheckAnalyze
59+
# Run static analysis tools (no security scan)
60+
./gradlew pmdMain checkstyleMain
6461

6562
# Verify code coverage meets minimum threshold
6663
./gradlew jacocoTestCoverageVerification
6764

6865
# Run individual analysis tools
69-
./gradlew spotbugsMain # Bug detection
7066
./gradlew pmdMain # Code style analysis
7167
./gradlew checkstyleMain # Style compliance
7268
```
@@ -75,8 +71,8 @@ java -jar build/libs/temporal-example-1.0.0.jar
7571

7672
This project includes comprehensive GitHub Actions workflows:
7773

78-
- **Build and Test** (`build.yml`): Tests on Java 17 & 21, builds artifacts
79-
- **Static Analysis** (`static-analysis.yml`): SpotBugs, PMD, Checkstyle, and security scanning
74+
- **Build and Test** (`build.yml`): Tests on Java 21, builds artifacts with Java 21
75+
- **Static Analysis** (`static-analysis.yml`): PMD and Checkstyle code quality analysis with Java 21
8076
- **Dependency Updates** (`dependabot.yml`): Automated dependency updates
8177

8278
## 🌐 Temporal Dashboard

0 commit comments

Comments
 (0)