Skip to content

Doc: mention elif, nested conditionals#12869

Open
mlibbey wants to merge 1 commit intoapache:masterfrom
mlibbey:hrw4udocelif
Open

Doc: mention elif, nested conditionals#12869
mlibbey wants to merge 1 commit intoapache:masterfrom
mlibbey:hrw4udocelif

Conversation

@mlibbey
Copy link
Contributor

@mlibbey mlibbey commented Feb 9, 2026

Missed some documentation when adding elif and nested conditionals.

Missed some documentation when adding elif and nested conditionals.
@mlibbey mlibbey added this to the 10.2.0 milestone Feb 9, 2026
@mlibbey mlibbey requested a review from zwoop February 9, 2026 17:34
@bryancall bryancall added the hrw4u label Feb 9, 2026
@bryancall bryancall requested a review from bneradt February 9, 2026 22:35
@bneradt bneradt dismissed their stale review February 10, 2026 21:59

Dismissing review - posted in error.

Copy link
Contributor

@bneradt bneradt left a comment

Choose a reason for hiding this comment

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

Looks good. A few minor comments.

}

READ_RESPONSE {
if inbound.status >= 400 {
Copy link
Contributor

Choose a reason for hiding this comment

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

If you look at the tools/hrw4u/grammar/hrw4u.g4, >= is not actually supported. Maybe we want to add this, but I don't think this will work as is.

@zwoop : thoughts?

Operators Raw text (`set-header`) Structured assignments or statements
Grouping `GROUP` / `GROUP:END` Use `()` inside `if` expressions
Else branches `else` with indented ops `else { ... }` with full block
Else branches `else` with indented ops `elif { ... }` and `else { ... }` with full blocks
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably good to elif () { ... }, with the () as you did above to indicate a condition should be present with elif. else is fine as is since it doesn't take a condition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants