Skip to content

CI check (fork-only test) - #1

Closed
DG47 wants to merge 1 commit into
masterfrom
fix/ci-maven-3.9-and-test-compile
Closed

DG47 wants to merge 1 commit into
masterfrom
fix/ci-maven-3.9-and-test-compile

Conversation

@DG47

@DG47 DG47 commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Fork-only run to verify the workflow change before opening upstream.

The ubuntu-24.04 runner image now ships Maven 3.10.0, which cannot build the
effective model of miredot-plugin 2.4.0: its toolsjar profile is no longer
activated, so the com.sun:tools systemPath stays ${toolsjar} and plugin
resolution fails before compilation. Install Maven 3.9.16 in the CI and release
workflows instead of using the runner's Maven.

CDMResultsAnalysisRunnerTest stubbed JdbcTemplate.query with any() as the
second argument, which javac 8 rejects as ambiguous between the Object[] and
PreparedStatementSetter overloads. Match PreparedStatementSetter, which is the
overload CDMResultsAnalysisRunner calls.
@DG47

DG47 commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

Verified; opened upstream.

@DG47 DG47 closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant