Skip to content

Upgrade minimum gcc version to 13, ruby to 3.0#439

Merged
jwillemsen merged 2 commits intoRemedyIT:masterfrom
jwillemsen:jwi-depgcc12andolder
Feb 5, 2026
Merged

Upgrade minimum gcc version to 13, ruby to 3.0#439
jwillemsen merged 2 commits intoRemedyIT:masterfrom
jwillemsen:jwi-depgcc12andolder

Conversation

@jwillemsen
Copy link
Member

@jwillemsen jwillemsen commented Feb 5, 2026

* .github/workflows/linux.yml:
* docs/src/getting_started.adoc:
* tests/tie/client.cpp:
* tests/tie/server.cpp:

Summary by CodeRabbit

  • Documentation

    • Updated minimum compiler requirement to GCC 13+ (previously 11+)
    • Updated Ruby version requirements to 3.0-3.1.2
    • Added documentation describing memory management improvements using modern C++ semantics (std::shared_ptr, std::weak_ptr, move semantics)
  • Chores

    • Updated CI workflow to target Ubuntu 24.04 with GCC 13/14 and newer Ruby versions
    • Simplified build test cases by removing conditional compilation directives

    * .github/workflows/linux.yml:
    * docs/src/getting_started.adoc:
    * tests/tie/client.cpp:
    * tests/tie/server.cpp:
    * .github/workflows/linux.yml:
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This PR updates CI configuration to target Ubuntu 24.04 with GCC 13+, updates documentation prerequisites accordingly, and removes conditional compilation guards from test code related to TIE specialization handling, simplifying the build requirements and test logic.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/linux.yml
Updated Linux CI matrix to target Ubuntu 24.04 with GCC 13/14, introduced make_flags parameters (--release, --debug), adjusted Ruby versions (3.0–3.2), and propagated make_flags to build step invocations; removed entries for older GCC/Ubuntu combinations.
Documentation Updates
docs/src/getting_started.adoc
Updated compiler prerequisites from GCC 11+ to 13+, tightened Ruby prerequisites from 2.5–3.1.2 to 3.0–3.1.2, and expanded memory management documentation for IDL-to-C++11 mapping to reflect std::shared_ptr/std::weak_ptr semantics.
Test Code Simplification
tests/tie/client.cpp, tests/tie/server.cpp
Removed TAOX11_LACKS_TIE_SPECIALIZATION conditional branch in client test; removed GCC-specific preprocessor guards in server test, making TIE specialization handling unconditional.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 Ubuntu's grown up, GCC too—
Thirteen's the new standard, shiny and new,
No more condition forks, guards shed away,
Simpler bindings—hooray, hooray!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: upgrading GCC from 11+ to 13+ and Ruby from 2.5 to 3.0, which are the primary updates across the documentation, workflows, and test files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@jwillemsen jwillemsen merged commit fc43d6f into RemedyIT:master Feb 5, 2026
14 checks passed
@jwillemsen jwillemsen deleted the jwi-depgcc12andolder branch February 5, 2026 13:49
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