Skip to content

Create gradle.properties in ProcessingPluginTest#1475

Draft
Stefterv wants to merge 4 commits intoprocessing:mainfrom
Stefterv:fix-failing-plugin-tests
Draft

Create gradle.properties in ProcessingPluginTest#1475
Stefterv wants to merge 4 commits intoprocessing:mainfrom
Stefterv:fix-failing-plugin-tests

Conversation

@Stefterv
Copy link
Member

@Stefterv Stefterv commented Mar 10, 2026

Add creation of sketch/gradle.properties in ProcessingPluginTest, writing processing.group derived from System.getProperty("project.group").replace(".java", ""). This ensures the test project has the expected processing.group property within the tests

Closes #1485

Add creation of sketch/gradle.properties in ProcessingPluginTest, writing processing.group derived from System.getProperty("project.group").replace(".java", ""). This ensures the test project has the expected processing.group property within the tests
@Stefterv Stefterv marked this pull request as draft March 17, 2026 06:40
@catilac
Copy link
Collaborator

catilac commented Mar 18, 2026

is this ready for review? or still draft?

@Stefterv
Copy link
Member Author

So far this makes sure the tests fail on CI and local, the solution for the failing test still needs to be implemented

Gate calls to signAllPublications() behind a project property so signing only occurs when explicitly enabled. The release workflow now exports ORG_GRADLE_PROJECT_signMavenPackages=true for relevant jobs, and build scripts were updated to check the property before signing (core, java libraries: dxf, io, net, pdf, serial, svg). The preprocessor signing check was also adjusted to look for the signing.signMavenPackages property. This lets CI enable signing when keys are available and avoids signing failures when signing credentials are not configured.
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.

Local tests failing

2 participants