Skip to content

Add JRuby support for gem build and release#249

Merged
yahonda merged 1 commit intorsim:masterfrom
yahonda:add-jruby-support
Apr 13, 2026
Merged

Add JRuby support for gem build and release#249
yahonda merged 1 commit intorsim:masterfrom
yahonda:add-jruby-support

Conversation

@yahonda
Copy link
Copy Markdown
Collaborator

@yahonda yahonda commented Apr 13, 2026

Summary

  • Add ruby-oci8 as a runtime dependency only for CRuby (was a dev dependency for all platforms)
  • Set gem platform to java on JRuby
  • Add JRuby build job to the release workflow so both CRuby and JRuby gems are built and pushed

Follows the same approach as rsim/oracle-enhanced@22f0e67.

Test plan

  • Verify gem build ruby-plsql.gemspec succeeds on both CRuby and JRuby
  • Verify the release workflow builds and pushes both gems

🤖 Generated with Claude Code

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds JRuby-specific gem packaging and updates the release workflow to build and publish both CRuby and JRuby variants, while making ruby-oci8 a CRuby-only runtime dependency.

Changes:

  • Update gemspec to set platform to java on JRuby and add ruby-oci8 as a CRuby runtime dependency.
  • Add a JRuby gem build job to the GitHub Actions release workflow.
  • Update the release job to download and push both built gem artifacts.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
ruby-plsql.gemspec Adds JRuby platform targeting and moves ruby-oci8 to a CRuby-only runtime dependency.
.github/workflows/release.yml Builds a JRuby gem artifact and pushes both CRuby + JRuby gems on tag releases.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yahonda yahonda merged commit 586ae11 into rsim:master Apr 13, 2026
22 checks passed
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.

2 participants