Skip to content

Commit e4c8690

Browse files
authored
Merge pull request #13 from braintrustdata/ark/code-cleanup-and-formatting
Ark/code cleanup and formatting
2 parents b4ec025 + 514fef0 commit e4c8690

88 files changed

Lines changed: 1985 additions & 10740 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/ci.yml

Lines changed: 10 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -8,56 +8,30 @@ on:
88

99
jobs:
1010
build:
11-
runs-on: ubuntu-latest
11+
# we want to run ubuntu-latest but we'll pin to a specific version so CI is reproducable
12+
runs-on: ubuntu-24.04
1213
strategy:
1314
matrix:
14-
java: [ '17', '21' ]
15-
15+
java: [ '17' ] # Java is
16+
1617
steps:
1718
- uses: actions/checkout@v4
18-
19+
1920
- name: Set up JDK ${{ matrix.java }}
2021
uses: actions/setup-java@v4
2122
with:
2223
java-version: ${{ matrix.java }}
2324
distribution: 'temurin'
24-
25+
2526
- name: Grant execute permission for gradlew
2627
run: chmod +x gradlew
27-
28-
- name: Check code formatting
29-
run: ./gradlew spotlessCheck
30-
31-
- name: Build with Gradle
32-
run: ./gradlew build
33-
34-
- name: Run tests
35-
run: ./gradlew test
36-
28+
29+
- name: Run all checks and tests
30+
run: ./gradlew compileTestJava check
31+
3732
- name: Upload test results
3833
uses: actions/upload-artifact@v4
3934
if: always()
4035
with:
4136
name: test-results-java-${{ matrix.java }}
4237
path: build/test-results/test/
43-
44-
lint:
45-
runs-on: ubuntu-latest
46-
47-
steps:
48-
- uses: actions/checkout@v4
49-
50-
- name: Set up JDK 17
51-
uses: actions/setup-java@v4
52-
with:
53-
java-version: '17'
54-
distribution: 'temurin'
55-
56-
- name: Grant execute permission for gradlew
57-
run: chmod +x gradlew
58-
59-
- name: Check code formatting
60-
run: ./gradlew spotlessCheck
61-
62-
- name: Run static analysis
63-
run: ./gradlew check

build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ plugins {
66
}
77

88
group = 'dev.braintrust'
9-
version = '0.1.0-SNAPSHOT'
9+
version = '0.0.1-SNAPSHOT'
1010

1111
java {
1212
sourceCompatibility = JavaVersion.VERSION_17

0 commit comments

Comments
 (0)