Skip to content

Conversation

@liamwhite
Copy link
Contributor

Just a quick thing, since these started getting flagged due to my editor config. If there are any controversial changes I can back them out and add to the ignore list instead.

Copy link
Owner

@kivikakk kivikakk left a comment

Choose a reason for hiding this comment

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

All LGTM! Thanks so much!

compare_strs(&output_from_rt, expected, "roundtrip", &md);
}

fn asssert_node_eq<'a>(node: Node<'a>, location: &[usize], expected: &NodeValue) {
Copy link
Owner

Choose a reason for hiding this comment

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

🐍

fn nul_at_eof() {
html("foo\0", "<p>foo\u{fffd}</p>\n");
html("foo\0ba", "<p>foo\u{fffd}ba</p>\n");
html("foo\0ba\0", "<p>foo\u{fffd}ba\u{fffd}</p>\n");
Copy link
Owner

Choose a reason for hiding this comment

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

🐑

@kivikakk kivikakk enabled auto-merge November 15, 2025 05:34
@kivikakk kivikakk merged commit 7d1b3c8 into kivikakk:main Nov 15, 2025
23 checks passed
@liamwhite liamwhite deleted the typos branch November 15, 2025 15:02
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.

2 participants