Skip to content

publication: route Plus 25.11 compatibility releases - #30

Merged
andrebrait merged 1 commit into
mainfrom
issue/2926-compat-25-11
Aug 31, 2026
Merged

andrebrait merged 1 commit into
mainfrom
issue/2926-compat-25-11

Conversation

@andrebrait

@andrebrait andrebrait commented Aug 31, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add the Plus 25.11 runtime tuple (FreeBSD 16.0-RELEASE, PHP 8.4, Python 3.11, ci: false) to the compatibility ROUTE matrix so tagged stable Releases without build handoffs can publish into stable/plus-25.11.

ci: false is inert to the publisher (routing keys on variant + version identity); the tagged-ingestion live gate's ci:false handling is a pfBlockerNG/pfBlockerNG change landing separately.

Verification

  • Frozen RED: test_compatibility_matrix_covers_every_production_route failed on ('Plus', '25.11') missing; GREEN with test hash ecacb8fa5541782a466de29fbb2027fca2f2ac3f unchanged.
  • tests/test_ingest_workflow.py — 13 passed.
  • Canonical jq . byte-identity for the matrix file.

Refs pfBlockerNG/pfBlockerNG#2926.

Authored with OMP; no configured GitHub-recognized OMP coauthor identity was available.

Summary by CodeRabbit

  • New Features

    • Added compatibility support for pfSense Plus 25.11, including FreeBSD 16.0, PHP 8.4, and the appropriate Python configuration.
    • Registered pfSense Plus 25.11 as an active production route.
  • Tests

    • Updated compatibility validation to include the pfSense Plus 25.11 route.

@coderabbitai

coderabbitai Bot commented Aug 31, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 016c8a0d-a4f2-4a47-bc2a-fbbb9b7e2e29

📥 Commits

Reviewing files that changed from the base of the PR and between 9a07988 and 453f639.

📒 Files selected for processing (2)
  • publication/compatibility-route-matrix.json
  • tests/test_ingest_workflow.py

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The compatibility matrix adds pfSense Plus 25.11 with FreeBSD 16.0, PHP 8.4, Python 3.11, and the pfsense-plus image. The production route test now expects this route.

Changes

Compatibility route coverage

Layer / File(s) Summary
pfSense Plus 25.11 route and contract test
publication/compatibility-route-matrix.json, tests/test_ingest_workflow.py
The matrix adds the active Plus 25.11 route with ci: false and no extra packages. The contract test includes the ("Plus", "25.11") pair.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 453f6

This localized compatibility-route update is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Poem

A rabbit checked the route today
Plus 25.11 joined the array
FreeBSD and PHP stand bright
Python hops into the matrix right
Empty packages, clean and light
The test now guards the path just right

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (1 skipped: 1 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the addition of the Plus 25.11 compatibility route. It is concise and directly related to the main change.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (1 skipped: 1 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch

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.

@andrebrait
andrebrait merged commit 7dfb0cd into main Aug 31, 2026
3 checks passed
@andrebrait
andrebrait deleted the issue/2926-compat-25-11 branch August 31, 2026 02:00
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