From 7647685d8c1cc506de8a3a11a3407e6f90ba29e8 Mon Sep 17 00:00:00 2001 From: jannetty Date: Sat, 11 Apr 2026 18:48:19 -0700 Subject: [PATCH 01/13] update build and CI configuration --- .github/config/checks.xml | 1 + build.gradle | 4 ++-- gradle/wrapper/gradle-wrapper.properties | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/config/checks.xml b/.github/config/checks.xml index cfd73a51d..c54318bb2 100644 --- a/.github/config/checks.xml +++ b/.github/config/checks.xml @@ -44,6 +44,7 @@ + diff --git a/build.gradle b/build.gradle index e34909fab..fdd0c5417 100644 --- a/build.gradle +++ b/build.gradle @@ -62,8 +62,8 @@ jacocoTestReport { }) } reports { - xml.enabled true - html.enabled true + xml.required.set(true) + html.required.set(true) } } diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties index a4b442974..48c0a02ca 100644 --- a/gradle/wrapper/gradle-wrapper.properties +++ b/gradle/wrapper/gradle-wrapper.properties @@ -1,5 +1,5 @@ distributionBase=GRADLE_USER_HOME distributionPath=wrapper/dists -distributionUrl=https\://services.gradle.org/distributions/gradle-6.3-bin.zip +distributionUrl=https\://services.gradle.org/distributions/gradle-8.7-bin.zip zipStoreBase=GRADLE_USER_HOME zipStorePath=wrapper/dists From 7b8321242d5c964c255c26f212b55521c99aa9b6 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:11:01 -0700 Subject: [PATCH 02/13] Added delta coverage changes to build.gradle and build.yml --- .github/workflows/build.yml | 22 ++++++++++++++++++++++ build.gradle | 12 ++++++++++++ 2 files changed, 34 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 7e60d007e..0a37370df 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -67,6 +67,28 @@ jobs: folder: pages/coverage target-folder: coverage + publish-delta-coverage: + runs-on: ubuntu-latest + + steps: + + - name: Checkout the repo + uses: actions/checkout@v4 + with: + fetch-depth: 0 + + - name: Set up JDK 11 + uses: actions/setup-java@v4 + with: + java-version: '11' + distribution: 'corretto' + + - name: Run delta coverage + run: ./gradlew test deltaCoverage + + - name: Publish summary + run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY + update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} diff --git a/build.gradle b/build.gradle index e34909fab..dba852759 100644 --- a/build.gradle +++ b/build.gradle @@ -3,6 +3,7 @@ plugins { id "jacoco" id "com.palantir.git-version" version "3.0.0" id "com.diffplug.spotless" version "6.25.0" + id "io.github.surpsg.delta-coverage" version "2.5.0" } version = "3.3.0" @@ -67,6 +68,17 @@ jacocoTestReport { } } +deltaCoverageReport { + diffSource { + diffSource.git.diffBase.set("refs/remotes/origin/main") + } + + reports { + html.set(true) + xml.set(true) + } +} + spotless { java { removeUnusedImports() From cd578de62f45e74bbe87b3c0e5127bca44a9edec Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:22:57 -0700 Subject: [PATCH 03/13] Added markdown file generation for summary --- build.gradle | 1 + 1 file changed, 1 insertion(+) diff --git a/build.gradle b/build.gradle index d0768bb0b..92c00c5fa 100644 --- a/build.gradle +++ b/build.gradle @@ -76,6 +76,7 @@ deltaCoverageReport { reports { html.set(true) xml.set(true) + markdown.set(true) } } From bd0ef3345ba2d987f7c2780c29707ae01124c912 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:36:38 -0700 Subject: [PATCH 04/13] Added documentation for summary --- .github/workflows/build.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 0a37370df..841364500 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -89,6 +89,9 @@ jobs: - name: Publish summary run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY + - name: Add disclaimer + run: cat "The summary table should list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY + update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From bb5836af1d703298f59c1ac920dd2e35111c4279 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:39:19 -0700 Subject: [PATCH 05/13] Fixed disclaimer printing command --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 841364500..1e9608cf1 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -90,7 +90,7 @@ jobs: run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY - name: Add disclaimer - run: cat "The summary table should list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY + run: echo "The summary table should list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From 737e98a289db3f0e63c4bea2a4df5346ef973eff Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:41:38 -0700 Subject: [PATCH 06/13] Fixed markdown for disclaimer method --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 1e9608cf1..3741f5a9e 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -90,7 +90,7 @@ jobs: run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY - name: Add disclaimer - run: echo "The summary table should list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY + run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run `./gradlew test deltaCoverage` locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From 29df525b5d945e9bb30cb634bc8b29105c4b6f0c Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:43:25 -0700 Subject: [PATCH 07/13] Removed broken markdown --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3741f5a9e..2abb8768c 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -90,7 +90,7 @@ jobs: run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY - name: Add disclaimer - run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run `./gradlew test deltaCoverage` locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY + run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From 1854b4da8a2c8b196a10da24e9f2a4d7a6a6a385 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 20 Apr 2026 12:46:14 -0700 Subject: [PATCH 08/13] Linked plugin repo --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 2abb8768c..9f92d2f74 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -90,7 +90,7 @@ jobs: run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY - name: Add disclaimer - run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY + run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin](https://github.com/gw-kit/delta-coverage-plugin)" >> $GITHUB_STEP_SUMMARY update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From 5660bc9747f697aa3bd1b1ca7782912900e180b3 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 27 Apr 2026 09:15:22 -0700 Subject: [PATCH 09/13] Test changes --- .../module/PottsModuleProliferationVolumeBasedDivision.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java index e154be0d4..1d051b099 100644 --- a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java +++ b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java @@ -41,5 +41,9 @@ public void step(MersenneTwisterFast random, Simulation sim) { if (sizeCheck) { addCell(random, sim); } + // TEST CHANGES: + if (true) { + System.out.println("test"); + } } } From eb545f52f92fa6cbe8d3d90364375da94bf747d6 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 27 Apr 2026 09:17:15 -0700 Subject: [PATCH 10/13] Test changes --- .../module/PottsModuleProliferationVolumeBasedDivision.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java index 1d051b099..7e98cbe65 100644 --- a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java +++ b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java @@ -43,7 +43,10 @@ public void step(MersenneTwisterFast random, Simulation sim) { } // TEST CHANGES: if (true) { - System.out.println("test"); + System.out.println("tested"); + } + if (false) { + System.out.println("untested"); } } } From 0fb50b5e2c36526124035c2adb98456dab625c51 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 27 Apr 2026 09:19:46 -0700 Subject: [PATCH 11/13] Test changes --- .../module/PottsModuleProliferationVolumeBasedDivision.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java index 7e98cbe65..5eacadf4c 100644 --- a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java +++ b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java @@ -45,7 +45,7 @@ public void step(MersenneTwisterFast random, Simulation sim) { if (true) { System.out.println("tested"); } - if (false) { + if (cell.getVolume() < 0) { System.out.println("untested"); } } From 10601ecd46c8818b3b7da7cf84ece939bfa14fdf Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 27 Apr 2026 09:34:50 -0700 Subject: [PATCH 12/13] Revert test commit This reverts commit 1854b4da8a2c8b196a10da24e9f2a4d7a6a6a385. --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 9f92d2f74..2abb8768c 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -90,7 +90,7 @@ jobs: run: cat build/reports/coverage-reports/delta-coverage/report.md >> $GITHUB_STEP_SUMMARY - name: Add disclaimer - run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin](https://github.com/gw-kit/delta-coverage-plugin)" >> $GITHUB_STEP_SUMMARY + run: echo "The summary table will list class files and percent changes on each row, with totals at the bottom. If you have no class file changes, your totals will show NaN%. Run ./gradlew test deltaCoverage locally to view line-by-line changes. [Plugin Documentation](https://gw-kit.github.io/delta-coverage-plugin/stable/)" >> $GITHUB_STEP_SUMMARY update-build-badges: if: ${{ always() && github.ref == 'refs/heads/main' }} From 526f9c747265df72562ceb3152852ce34aca6664 Mon Sep 17 00:00:00 2001 From: Alice Fu Date: Mon, 27 Apr 2026 09:36:26 -0700 Subject: [PATCH 13/13] Removed test changes --- .../PottsModuleProliferationVolumeBasedDivision.java | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java index 5eacadf4c..e154be0d4 100644 --- a/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java +++ b/src/arcade/potts/agent/module/PottsModuleProliferationVolumeBasedDivision.java @@ -41,12 +41,5 @@ public void step(MersenneTwisterFast random, Simulation sim) { if (sizeCheck) { addCell(random, sim); } - // TEST CHANGES: - if (true) { - System.out.println("tested"); - } - if (cell.getVolume() < 0) { - System.out.println("untested"); - } } }