-
Notifications
You must be signed in to change notification settings - Fork 68
Upgrade base OS and tooling #656
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Java 1.7 is no longer supported
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughThe changes primarily involve formatting and modernization across build scripts, Dockerfiles, macros, and test files. Compiler and coverage tool versioning is updated, especially for Clang, GCC, and gcov tools, with corresponding CMake and shell script adjustments. Several macros and array initializations are reformatted for conciseness. Dockerfiles are updated to use Ubuntu 24.04 and newer Clang versions. Minor code formatting and include adjustments are made in source and test files. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant BuildSystem (CMake)
participant Compiler (GCC/Clang)
participant CoverageTool (gcov/llvm-cov)
participant Docker
participant TestRunner
User->>BuildSystem: Configure build (with coverage)
BuildSystem->>Compiler: Set coverage flags (version-specific)
Compiler-->>BuildSystem: Compile with proper flags
User->>Docker: Build Docker image
Docker->>BuildSystem: Use Ubuntu 24.04, Clang 18, etc.
User->>CoverageTool: Run coverage script
CoverageTool->>Compiler: Detect version
CoverageTool->>CoverageTool: Use version-specific gcov/llvm-cov
User->>TestRunner: Run tests (with updated macros, includes, formatting)
TestRunner-->>User: Report results
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
🎉 Thank you for your code contribution! To guarantee the change/addition is conformant to the OCF Specification, we would like to ask you to execute OCF Conformance Testing of your change ☝️ when your work is ready to be reviewed. ℹ️ To verify your latest change (5168007), label this PR with |
|



No description provided.