Skip to content

Conversation

@nelsonjchen
Copy link

@nelsonjchen nelsonjchen commented Nov 2, 2025

Summary

<code> doesn't seem to work over multiple lines

broken during a reformatting refactor?

933987a#diff-fa5863d597bafb036dee58cc25456f938afcf47db163e9ce658b76fb040b136cR173

Screenshots (optional)

image

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

`<code>` doesn't seem to work over multiple lines 

broken during a reformatting refactor?
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the formatting of the setInnerContent method documentation in the HTML Rewriter API reference by replacing a multi-line <code> block with a cleaner inline code format.

  • Simplified the markdown formatting for the setInnerContent method signature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- <code>
setInnerContent(contentContent, contentOptionsContentOptionsoptional)
</code>
- `setInnerContent(contentContent, contentOptionsContentOptionsoptional)`
Copy link

Copilot AI Nov 2, 2025

Choose a reason for hiding this comment

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

The parameter names appear to have duplicated words. 'contentContent' should likely be 'content' and 'contentOptionsContentOptionsoptional' should likely be 'contentOptions' (optional).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants