Skip to content

Feb16-release#28316

Merged
fb1337 merged 4 commits intocloudflare:productionfrom
rag-cf:feb16-release
Feb 18, 2026
Merged

Feb16-release#28316
fb1337 merged 4 commits intocloudflare:productionfrom
rag-cf:feb16-release

Conversation

@rag-cf
Copy link
Contributor

@rag-cf rag-cf commented Feb 13, 2026

Add rules for Feb16 release

@rag-cf rag-cf requested review from a team, fb1337 and vs-mg as code owners February 13, 2026 11:31
@rag-cf rag-cf marked this pull request as draft February 13, 2026 11:31
@irvinebroque
Copy link
Contributor

/bonk recommend improvements to this changelog

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

The content looks good — the frontmatter, component import, and CVE descriptions are all correct. I noticed one category of issue that should be addressed: inconsistent indentation in the HTML table.

Inconsistent indentation in <tbody> section

Comparing against existing WAF changelog entries (e.g., 2025-11-17-waf-release.mdx, 2025-11-03-waf-release.mdx), the established convention uses consistent tab indentation throughout the entire table. The new file mixes tabs and spaces, and has misaligned elements within <tbody>.

Specifically:

  • <tbody> on line 29 uses spaces instead of a tab
  • <tr>, <td>, and </tr> elements inside <tbody> use spaces with inconsistent levels
  • The first </tr> (line 39) uses 2-space indent while the second </tr> (line 50) uses a tab+spaces mix

The indentation should match the convention used in the <thead> section of the same file and in all other WAF changelogs: tab-based indentation with consistent nesting depth.

Note: pedrosousa's suggestion on line 50 for the closing </tr> tag fixes the immediate build issue there, but the rest of the <tbody> still has mixed indentation that should ideally be cleaned up for consistency.

@github-actions
Copy link
Contributor

Review posted on PR #28316. Here's a summary of what I found:

Content: The frontmatter, component import, CVE descriptions, and table data are all correct and follow the established WAF changelog pattern.

Issue found: The <tbody> section (lines 29-51) has inconsistent indentation -- it mixes tabs and spaces, while the <thead> section and all other WAF changelog files use consistent tab-based indentation. I posted an inline suggestion that normalizes the entire <tbody> block to use tabs matching the repository convention. This suggestion also subsumes pedrosousa's earlier fix for the </tr> tag on line 50.

github run

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@rag-cf rag-cf marked this pull request as ready for review February 18, 2026 00:59
@rag-cf rag-cf requested a review from pedrosousa February 18, 2026 07:52
@pedrosousa
Copy link
Contributor

@rag-cf Do we need to update the Scheduled changes page to remove the two entries?

@fb1337 fb1337 merged commit 11656e9 into cloudflare:production Feb 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants