Skip to content

fixes #186 remove gradle build - #187

Merged
stevehu merged 1 commit into
masterfrom
issue186
Oct 2, 2026
Merged

stevehu merged 1 commit into
masterfrom
issue186

Conversation

@stevehu

@stevehu stevehu commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings October 2, 2026 20:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

Gradle artifacts and callers are removed, while the Maven native-build replacement has focused packaging and failure coverage.

Review effort: Balanced
Findings: None

What changed in this PR

Removes the obsolete Gradle build path identified in #186 and standardizes native packaging on Maven and Docker.

Changes:

  • Removes Gradle configuration, wrappers, scripts, and Eclipse integration.
  • Modernizes native-image packaging and bootstrap portability.
  • Documents and tests native build and failure handling.
File Description
tests/​test_native_image_script.py Adds native packaging regression tests.
README.md Documents Maven and native builds.
native-image-linux.sh Reworks Maven-driven native packaging.
Makefile Removes the Gradle build target.
gradlew.bat Removes the Windows Gradle wrapper.
gradlew Removes the Unix Gradle wrapper.
gradle/​wrapper/​gradle-wrapper.properties Removes Gradle wrapper configuration.
gradle.properties Removes obsolete Gradle dependencies.
build.gradle Removes the Gradle build definition.
build_graalvm.sh Removes the Gradle-specific native builder.
bootstrap Makes the launcher POSIX-compatible.
.project Removes Eclipse Gradle integration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@DiogoFKT DiogoFKT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@stevehu
stevehu merged commit 5f27256 into master Oct 2, 2026
1 check passed
@stevehu
stevehu deleted the issue186 branch October 2, 2026 21:09
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.

3 participants