chore: add LLVM version support policy#87
Conversation
Define a three-tier version support policy covering: - Tier 1 (Full): latest 3 major versions with full CI & binary support - Tier 2 (Standard): versions from the support floor (LLVM 11) to Tier 1 - Tier 3 (Legacy): versions below the floor, no default support Includes special support channels (community/sponsored/paid) and build-from-source instructions for legacy versions.
WalkthroughA new ChangesLLVM Version Support Policy
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
VERSION_SUPPORT_POLICY.md (1)
57-59: 💤 Low valueRepetitive sentence openings in Tier 3 bullet list.
The three bullet points for Tier 3 support (Lines 57–59) all begin with "No," which reads as repetitive. As a minor style improvement, consider rewording one or more to vary the structure while retaining clarity:
- **No pre-built binaries are published.** - Docker images are not maintained. - No CI testing.Or:
- **No pre-built binaries are published.** - **No Docker images are maintained.** - **No CI testing is performed.**🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@VERSION_SUPPORT_POLICY.md` around lines 57 - 59, The Tier 3 support bullet list contains repetitive sentence openings with all three points beginning with "No". Reword at least one or more of the bullet points to vary the sentence structure while maintaining clarity. You may use the suggested alternatives provided in the comment (such as changing "No Docker images are maintained" to "Docker images are not maintained," or ensuring consistent formatting and complete sentences across all three bullets) or apply your own variation that reduces the repetition while keeping the meaning clear.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@VERSION_SUPPORT_POLICY.md`:
- Line 38: The VERSION_SUPPORT_POLICY.md file contains an inconsistency across
multiple locations regarding the default version. Line 38 and Line 189 claim the
default version "tracks the middle Tier-1 version," while Line 76 sets LLVM 20
as the default. However, if Tier 1 is defined as versions 22, 21, 20 (newest to
oldest), then 21 is the numerically middle version, not 20. You must resolve
this by either: (1) changing the language at Lines 38 and 189 to accurately
describe 20 (such as "oldest Tier-1 version" or "minimum Tier-1 version for
maximum stability") to clarify the actual intent, or (2) if the default should
instead be 21, update Line 76 to mark LLVM 21 as the default and correct the
version references in Lines 38 and 189 from 20 to 21. Ensure all three locations
use consistent terminology and version numbers.
---
Nitpick comments:
In `@VERSION_SUPPORT_POLICY.md`:
- Around line 57-59: The Tier 3 support bullet list contains repetitive sentence
openings with all three points beginning with "No". Reword at least one or more
of the bullet points to vary the sentence structure while maintaining clarity.
You may use the suggested alternatives provided in the comment (such as changing
"No Docker images are maintained" to "Docker images are not maintained," or
ensuring consistent formatting and complete sentences across all three bullets)
or apply your own variation that reduces the repetition while keeping the
meaning clear.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b1695e52-6bf6-490a-a05e-93b3d6371a3b
📒 Files selected for processing (1)
VERSION_SUPPORT_POLICY.md
Summary
This PR introduces
VERSION_SUPPORT_POLICY.md— a formal, three-tier policy for which LLVM/Clang versions cpp-linter supports across all projects.Three-Tier System
What the Document Covers
Design Decisions
Closes #N/A (new policy, no linked issue)
Summary by CodeRabbit
Release Notes
Documentation