chore: update EOL and open changes - #288
Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe README release table now includes RH 6.6 and marks RH 6.4 as EOL. The patch list now contains six entries for recent fixes and metrics changes. ChangesREADME updates
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ 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 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcantrill The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@README.md`:
- Line 37: Add one blank line after the final patch-table row and before the “##
Issues” heading in the README, leaving the table content unchanged.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| | | [Add truncate option for oversized events](https://github.com/vectordotdev/vector/pull/25567) | v0.54-rh | [Pending](https://github.com/vectordotdev/vector/pull/25567) | | ||
| | LOG-9375 | [Enhance the prometheus exporter with SAR auth strategy](https://github.com/ViaQ/vector/pull/265) | v0.54-rh | [Pending](https://github.com/vectordotdev/vector/pull/25412) | | ||
| | LOG-9560 | [Fix regression for UDP socket recovery after error](https://github.com/ViaQ/vector/pull/281) | v0.54-rh | [Pending](https://github.com/vectordotdev/vector/pull/25844) | | ||
| | | [Fix AWS sinks to emit component_sent_bytes_total with component labels](https://github.com/ViaQ/vector/pull/285) | v0.54-rh | [Pending](https://github.com/vectordotdev/vector/pull/25915) | |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add a blank line after the patch table.
## Issues follows the final table row without a blank line. Add one blank line after Line 37 to resolve markdownlint MD058.
As per path instructions, “Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity”; this comment is limited to the reported lint failure.
🧰 Tools
🪛 markdownlint-cli2 (0.23.2)
[warning] 37-37: Tables should be surrounded by blank lines
(MD058, blanks-around-tables)
🤖 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 `@README.md` at line 37, Add one blank line after the final patch-table row and
before the “## Issues” heading in the README, leaving the table content
unchanged.
Sources: Path instructions, Linters/SAST tools
1 participant
cc @Clee2691 @vparfonov