Skip to content

Fix typo in block_errors documentation#10591

Merged
maskit merged 1 commit intoapache:masterfrom
maskit:fix_block_error_doc
Oct 12, 2023
Merged

Fix typo in block_errors documentation#10591
maskit merged 1 commit intoapache:masterfrom
maskit:fix_block_error_doc

Conversation

@maskit
Copy link
Copy Markdown
Member

@maskit maskit commented Oct 12, 2023

if (tag == "block_errors.enabled") {

@maskit maskit added this to the 10.0.0 milestone Oct 12, 2023
@maskit maskit self-assigned this Oct 12, 2023
Copy link
Copy Markdown
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 fine to me. Unless we want to switch things the other way (i.e., we should use enable instead of enabled). But I think that enabled is more typical. So I say go forward with your patch as is.

@maskit maskit merged commit 641704b into apache:master Oct 12, 2023
bryancall pushed a commit that referenced this pull request Oct 15, 2023
@bryancall bryancall modified the milestones: 10.0.0, 9.2.4 Oct 15, 2023
@bryancall
Copy link
Copy Markdown
Contributor

@maskit We should have a PR for 8.1.x separately so the change can be part of the changelog and tied to 8.1.10. I cherry-picked this change into 9.2.x for the next 9.2.4 release.

maskit added a commit to maskit/trafficserver that referenced this pull request Oct 16, 2023
(cherry picked from commit 641704b)

 Conflicts:
	doc/admin-guide/plugins/block_errors.en.rst
@maskit
Copy link
Copy Markdown
Member Author

maskit commented Oct 16, 2023

#10618

bryancall pushed a commit that referenced this pull request Oct 16, 2023
(cherry picked from commit 641704b)

 Conflicts:
	doc/admin-guide/plugins/block_errors.en.rst
traeak pushed a commit to traeak/trafficserver that referenced this pull request Dec 15, 2023
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.

3 participants